Skip to content

Comments

nix: update overlay version since we updated rust version#2828

Open
justus-camp-microsoft wants to merge 1 commit intomicrosoft:mainfrom
justus-camp-microsoft:fix_nix_shell
Open

nix: update overlay version since we updated rust version#2828
justus-camp-microsoft wants to merge 1 commit intomicrosoft:mainfrom
justus-camp-microsoft:fix_nix_shell

Conversation

@justus-camp-microsoft
Copy link
Contributor

When we updated Rust to 1.93, the oxalica rust overlay that Nix uses wasn't updated, which means the overlay doesn't have the right version of rust to use. This updates the commit for the overlay and also adds some instructions to figure out how to do so.

@justus-camp-microsoft justus-camp-microsoft requested a review from a team as a code owner February 23, 2026 23:48
Copilot AI review requested due to automatic review settings February 23, 2026 23:48
@github-actions github-actions bot added the Guide label Feb 23, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s Nix development environment to match the workspace Rust toolchain version by bumping the pinned oxalica/rust-overlay commit, and documents the process for updating it when rust-version changes.

Changes:

  • Bump the pinned rust-overlay commit + content hash in shell.nix so Nix can resolve Rust 1.93.
  • Add developer documentation describing how to update the rust-overlay pin and hash when Cargo.toml’s rust-version is bumped.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
shell.nix Updates the pinned rust-overlay tarball commit + sha256 to include Rust 1.93.
Guide/src/dev_guide/dev_tools/flowey/nix.md Documents the steps to update the rust-overlay pin/hash when Rust is bumped.

shell.nix Outdated
@@ -6,8 +6,8 @@ let
};
# Pinned rust-overlay from stable branch
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we add a comment here with the current Rust version number (1.93)? That way I'll find it when updating, since I usually just do a ctrl F for the version number lol.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The rust version is in the comment now - let me know if you need anything else to make this easier since you're usually the one that updates our rust version

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants