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

Fix CI with the latest rustup #2052

Merged
merged 2 commits into from
Jun 6, 2023

Conversation

asomers
Copy link
Member

@asomers asomers commented Jun 5, 2023

No description provided.

@asomers asomers mentioned this pull request Jun 5, 2023
@asomers asomers changed the title DO NOT MERGE try to fix the Rust Stable build Fix CI with the latest rustup Jun 5, 2023
asomers and others added 2 commits June 5, 2023 18:08
The latest Rustup does not allow the toolchain specification to be
blank, which we were using.  Fix it by ensuring that only one toolchain
is ever installed in a given task, so we won't need to use the
+$TOOLCHAIN with cargo.
fix the new clippy::non-minimal-cfg lint
@asomers asomers force-pushed the toolchain-names-must-not-be-empty branch from d1f82c9 to 57cdbed Compare June 6, 2023 00:11
@asomers
Copy link
Member Author

asomers commented Jun 6, 2023

bors r+

@bors bors bot merged commit 89b4976 into nix-rust:master Jun 6, 2023
@asomers asomers deleted the toolchain-names-must-not-be-empty branch June 6, 2023 03:50
asomers added a commit to asomers/nix that referenced this pull request Aug 27, 2023
2052: Fix CI with the latest rustup r=asomers a=asomers

Co-authored-by: Alan Somers <asomers@gmail.com>
Co-authored-by: Ben Kimock <kimockb@gmail.com>
asomers added a commit to asomers/nix that referenced this pull request Aug 27, 2023
2052: Fix CI with the latest rustup r=asomers a=asomers

Co-authored-by: Alan Somers <asomers@gmail.com>
Co-authored-by: Ben Kimock <kimockb@gmail.com>
asomers added a commit to asomers/nix that referenced this pull request Aug 27, 2023
2052: Fix CI with the latest rustup r=asomers a=asomers

Co-authored-by: Alan Somers <asomers@gmail.com>
Co-authored-by: Ben Kimock <kimockb@gmail.com>
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