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.

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

Add pinned Rust version via rust-toolchain.toml #7546

Closed
wants to merge 1 commit into from

Conversation

kl
Copy link
Contributor

@kl kl commented Jan 29, 2025

As part of enabling reproducible builds we need to build a given commit with a pinned Rust version.

This commit adds a minimal rust-toolchain.toml file that specifies this version (this Rust version is enforced by rustup when invoking a cargo command).

In the container we extract the version that is specified in rust-toolchain.toml to ensure that the container is pre-built with the correct Rust version (so that when buildling in the container we do not need to download the right Rust version).

Related: #5680


This change is Reviewable

As part of enabling reproducible builds we need to build a given commit
with a pinned Rust version.

This commit adds a minimal rust-toolchain.toml file that specifies this
version (this Rust version is enforced by rustup when invoking a cargo
command).

In the container we extract the version that is specified in
rust-toolchain.toml to ensure that the container is pre-built with the
correct Rust version (so that when buildling in the container we do not
need to download the right Rust version).
@kl kl requested review from faern and albin-mullvad January 29, 2025 14:56
Copy link

linear bot commented Jan 29, 2025

@kl
Copy link
Contributor Author

kl commented Jan 29, 2025

We shall update and merge #5680 instead

@kl kl closed this Jan 29, 2025
@kl kl deleted the pin-rust-version-with-toolchain-file-des-554 branch January 29, 2025 16:08
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.

1 participant