From 836e6d3e021b56c84de76ff32f751ca4ea994079 Mon Sep 17 00:00:00 2001 From: figsoda Date: Tue, 19 Oct 2021 19:59:28 -0400 Subject: [PATCH 1/2] buildRustPackage: remove unused arguments, minor styling changes --- pkgs/build-support/rust/default.nix | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/pkgs/build-support/rust/default.nix b/pkgs/build-support/rust/default.nix index 2eb45bcafa13..840abb2d918a 100644 --- a/pkgs/build-support/rust/default.nix +++ b/pkgs/build-support/rust/default.nix @@ -1,17 +1,15 @@ -{ stdenv -, lib -, buildPackages +{ lib +, importCargoLock +, fetchCargoTarball +, rust +, stdenv +, callPackage , cacert +, git , cargoBuildHook , cargoCheckHook , cargoInstallHook , cargoSetupHook -, fetchCargoTarball -, importCargoLock -, rustPlatform -, callPackage -, git -, rust , rustc , libiconv , windows @@ -88,7 +86,7 @@ let (lib.removeSuffix ".json" (builtins.baseNameOf "${target}")) else target; - sysroot = (callPackage ./sysroot {}) { + sysroot = callPackage ./sysroot { } { inherit target shortTarget; RUSTFLAGS = args.RUSTFLAGS or ""; originalCargoToml = src + /Cargo.toml; # profile info is later extracted From f441e6b7f4574c85eb190b52390bbe61f355b514 Mon Sep 17 00:00:00 2001 From: figsoda Date: Mon, 18 Oct 2021 19:21:43 -0400 Subject: [PATCH 2/2] buildRustPackage,fetchCargoTarball: accept empty hashes --- pkgs/build-support/rust/default.nix | 16 ++++++---------- pkgs/build-support/rust/fetchCargoTarball.nix | 6 ++---- 2 files changed, 8 insertions(+), 14 deletions(-) diff --git a/pkgs/build-support/rust/default.nix b/pkgs/build-support/rust/default.nix index 840abb2d918a..3d7057dd7d97 100644 --- a/pkgs/build-support/rust/default.nix +++ b/pkgs/build-support/rust/default.nix @@ -17,12 +17,6 @@ { name ? "${args.pname}-${args.version}" - # SRI hash -, cargoHash ? "" - - # Legacy hash -, cargoSha256 ? "" - # Name for the vendored dependencies tarball , cargoDepsName ? name @@ -54,7 +48,7 @@ , buildAndTestSubdir ? null , ... } @ args: -assert cargoVendorDir == null && cargoLock == null -> cargoSha256 == "" && cargoHash == "" +assert cargoVendorDir == null && cargoLock == null -> !(args ? cargoSha256) && !(args ? cargoHash) -> throw "cargoSha256, cargoHash, cargoVendorDir, or cargoLock must be set"; assert buildType == "release" || buildType == "debug"; @@ -66,15 +60,17 @@ let else fetchCargoTarball ({ inherit src srcs sourceRoot unpackPhase cargoUpdateHook; name = cargoDepsName; - hash = cargoHash; patches = cargoPatches; - sha256 = cargoSha256; + } // lib.optionalAttrs (args ? cargoHash) { + hash = args.cargoHash; + } // lib.optionalAttrs (args ? cargoSha256) { + sha256 = args.cargoSha256; } // depsExtraArgs) else null; # If we have a cargoSha256 fixed-output derivation, validate it at build time # against the src fixed-output derivation to check consistency. - validateCargoDeps = !(cargoHash == "" && cargoSha256 == ""); + validateCargoDeps = args ? cargoHash || args ? cargoSha256; target = rust.toRustTargetSpec stdenv.hostPlatform; targetIsJSON = lib.hasSuffix ".json" target; diff --git a/pkgs/build-support/rust/fetchCargoTarball.nix b/pkgs/build-support/rust/fetchCargoTarball.nix index 2e8830b47574..c2be9aac82df 100644 --- a/pkgs/build-support/rust/fetchCargoTarball.nix +++ b/pkgs/build-support/rust/fetchCargoTarball.nix @@ -22,15 +22,13 @@ in , srcs ? [] , patches ? [] , sourceRoot ? "" -, hash ? "" -, sha256 ? "" , cargoUpdateHook ? "" , ... } @ args: let hash_ = - if hash != "" then { outputHashAlgo = null; outputHash = hash; } - else if sha256 != "" then { outputHashAlgo = "sha256"; outputHash = sha256; } + if args ? hash then { outputHashAlgo = null; outputHash = args.hash; } + else if args ? sha256 then { outputHashAlgo = "sha256"; outputHash = args.sha256; } else throw "fetchCargoTarball requires a hash for ${name}"; in stdenv.mkDerivation ({ name = "${name}-vendor.tar.gz";