-
Notifications
You must be signed in to change notification settings - Fork 48
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
ci: pin cross and rust versions #200
Conversation
Signed-off-by: jiaxiao zhou <[email protected]>
Signed-off-by: jiaxiao zhou <[email protected]>
Confirmed that an older cross image |
It looks like the |
cross-rs/cross#1418 this PR adds PIE back. |
Signed-off-by: jiaxiao zhou <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work locking in the version to avoid the PIE compilation issue.
Thanks! Just to clarify, locking in the version did not fix the PIE compilation issue. The fix was on upstream cross repo to enable PIE compilation in musl library. |
* ci: pin cross and rust versions * fixed the commit rev * pin rust version to 1.75 Signed-off-by: jiaxiao zhou <[email protected]>
This pull request includes updates to the Rust toolchain and the Makefile to ensure specific versions are used and to include new components and targets. The most important changes include updating the
rust-toolchain.toml
file to use Rust version 1.75.0 and updating theinstall-cross
target in theMakefile
to specify a specific commit when installingcross
.