Skip to content

Support ubuntu 22.04 in released binaries #78

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.

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

Closed
wants to merge 1 commit into from

Conversation

rukai
Copy link

@rukai rukai commented Feb 25, 2025

Currently releases are compiled on the latest ubuntu release on github actions, currently this is ubuntu 24.04 but that will change in the future as new releases are made available.
This PR changes releases to be specifically built on ubuntu 22.04.

As a result of this change, cargo-codspeed binary releases will be compatible with both ubuntu 22.04 and ubuntu 24.04.
Whereas previously cargo-codspeed was only compatible with ubuntu 24.04.

This is because glibc is linked to the final binary and glibc is compatible only with the versions newer or the same as the version on the system it was built with.

Here is an example of the error hit when running on ubuntu 22.04:
image

As a workaround I am compiling cargo-codspeed from source, but it would be a nice boost to CI completion time if we could instead make use of the precompiled binaries.

@GuillaumeLagrange
Copy link
Contributor

Hello, thanks for reporting the issue and for the PR !
We have addressed the root issue (#79), in a slightly different way here #80, closing this PR.
We will be releasing a new version with this fix very soon.

@rukai
Copy link
Author

rukai commented Feb 25, 2025

Musl builds will solve this issue, thanks.

@rukai
Copy link
Author

rukai commented Feb 25, 2025

However I believe #79 is a seperate issue, or at least I'm not sure how swapping to musl resolves it.

@GuillaumeLagrange
Copy link
Contributor

Thanks for pointing this out, we have reopened #79 and answered on the dedicated issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants