Skip to content

[nix] Fix OSS formal nix env - #2494

Merged
SamuelRiedel merged 2 commits into
lowRISC:masterfrom
SamuelRiedel:nix-formal
Sep 8, 2026
Merged

[nix] Fix OSS formal nix env#2494
SamuelRiedel merged 2 commits into
lowRISC:masterfrom
SamuelRiedel:nix-formal

Conversation

@SamuelRiedel

@SamuelRiedel SamuelRiedel commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Fix the failing OSS Formal flow in CI:

Signed-off-by: Samuel Riedel <sriedel@lowrisc.org>
nixpkgs needs to be updated to fix fetching from crates.io:
rust-lang/crates.io#13482

gomod2nix needs to be updated to prevent using deprecated features.

Signed-off-by: Samuel Riedel <sriedel@lowrisc.org>
@SamuelRiedel SamuelRiedel changed the title [nix] Replace mkshell-minimal [nix] Fix OSS nix env Sep 7, 2026
@SamuelRiedel SamuelRiedel changed the title [nix] Fix OSS nix env [nix] Fix OSS formal nix env Sep 7, 2026
@SamuelRiedel
SamuelRiedel marked this pull request as ready for review September 7, 2026 15:37

@hcallahan-lowrisc hcallahan-lowrisc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Sam! Assuming the CI job now passes, LGTM.

@SamuelRiedel
SamuelRiedel added this pull request to the merge queue Sep 8, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a manual request Sep 8, 2026
@SamuelRiedel
SamuelRiedel deleted the nix-formal branch September 8, 2026 10:03
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 8, 2026
@SamuelRiedel
SamuelRiedel restored the nix-formal branch September 8, 2026 10:07
@SamuelRiedel SamuelRiedel reopened this Sep 8, 2026
@SamuelRiedel
SamuelRiedel added this pull request to the merge queue Sep 8, 2026
Merged via the queue into lowRISC:master with commit 405c6d1 Sep 8, 2026
18 checks passed
@SamuelRiedel
SamuelRiedel deleted the nix-formal branch September 8, 2026 12:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants