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.4.1 -> 0.4.2 #75052
cargo-edit: 0.4.1 -> 0.4.2 #75052
Conversation
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.
LGTM
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.
LGTM.
Huh, so this didn't build for me on macOS and I had to diff --git a/pkgs/tools/package-management/cargo-edit/default.nix b/pkgs/tools/package-management/cargo-edit/default.nix
index 51c6def02c1..bad92e2c1a1 100644
--- a/pkgs/tools/package-management/cargo-edit/default.nix
+++ b/pkgs/tools/package-management/cargo-edit/default.nix
@@ -1,6 +1,6 @@
{ stdenv, lib, darwin
, rustPlatform, fetchFromGitHub
-, openssl, pkgconfig }:
+, openssl, pkgconfig, libiconv }:
rustPlatform.buildRustPackage rec {
pname = "cargo-edit";
@@ -16,7 +16,7 @@ rustPlatform.buildRustPackage rec {
cargoSha256 = "0prd53p20cha2y2qp8dmq0ywd32f6jm8mszdkbi4x606dj9bcgbl";
nativeBuildInputs = lib.optional (!stdenv.isDarwin) pkgconfig;
- buildInputs = lib.optional (!stdenv.isDarwin) openssl;
+ buildInputs = [ openssl ] ++ lib.optionals stdenv.isDarwin [ libiconv ];
propagatedBuildInputs = lib.optional stdenv.isDarwin darwin.apple_sdk.frameworks.Security;
meta = with lib; { Can someone verify that this isn't just my broken setup? |
@killercup ofBorg had the same issue. I'll patch it now. |
a73acb7
to
aa7e6b0
Compare
aa7e6b0
to
0b5751c
Compare
@GrahamcOfBorg build cargo-edit |
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.
Built locally and tested binaries, LGTM
Motivation for this change
https://github.com/killercup/cargo-edit/releases/tag/v0.4.2
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @Gerschtli @jb55 @oxalica
This change is