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
cargo-edit: 0.2.0 -> 0.3.0 #46100
cargo-edit: 0.2.0 -> 0.3.0 #46100
Conversation
oh, 3000 lines nix code for a single package? |
Does it also compile with |
Saving 3000 lines of Nix code.
Does it also compile with `buildRustPackage`?
Seems to!
|
@GrahamcOfBorg build cargo-edit |
|
||
propagatedBuildInputs = stdenv.lib.optionals stdenv.isDarwin [ darwin.apple_sdk.frameworks.Security ]; | ||
propagatedBuildInputs = lib.optional stdenv.isDarwin darwin.apple_sdk.frameworks.Security; |
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.
On linux cargo-edit
also needs pkgconfig
in nativeBuildInputs
and openssl
in buildInputs
.
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.
Any reason why darwin.apple_sdk.frameworks.Security
is not in buildInputs
?
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.
Not that I can see. Was like that before.
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.
darwin.apple_sdk.frameworks.Security
is probably fine. pkgs/build-support/rust/default-crate-overrides.nix
does the same.
Failure on x86_64-linux (full log) Attempted: cargo-edit Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: cargo-edit Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: cargo-edit Partial log (click to expand)
|
@GrahamcOfBorg build cargo-edit |
Success on x86_64-linux (full log) Attempted: cargo-edit Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: cargo-edit Partial log (click to expand)
|
Thanks! |
Success on x86_64-darwin (full log) Attempted: cargo-edit Partial log (click to expand)
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)