Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI][RISCV64] Try to use bookworm debian instead of sid #3992

Merged
merged 1 commit into from
Jun 13, 2023
Merged

[CI][RISCV64] Try to use bookworm debian instead of sid #3992

merged 1 commit into from
Jun 13, 2023

Conversation

ptitSeb
Copy link
Contributor

@ptitSeb ptitSeb commented Jun 13, 2023

Try to use bookworm debian instead of sid

theduke
theduke approved these changes Jun 13, 2023
View reviewed changes
Copy link
Contributor

@theduke theduke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is one potential concern here:

Is bookworm on a newer libc version?
If so, it would probably mean our pre-built wasmer for releases won't be compatible with older distros?

@ptitSeb
Copy link
Contributor Author

ptitSeb commented Jun 13, 2023

There is one potential concern here:

Is bookworm on a newer libc version? If so, it would probably mean our pre-built wasmer for releases won't be compatible with older distros?

Bookworm is basically the 1st stable Debian version for RISCV64. We were using unstable sid before, so switching to stable is an improvment and staying on unstable or testing is a lesser choice imo

@ptitSeb ptitSeb merged commit 8cd03c8 into master Jun 13, 2023
@ptitSeb ptitSeb deleted the ci_riscv64_update branch June 13, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@theduke theduke theduke approved these changes

@Michael-F-Bryan Michael-F-Bryan Awaiting requested review from Michael-F-Bryan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ptitSeb @theduke