diff options
author | Profpatsch <mail@profpatsch.de> | 2021-04-24T11·32+0200 |
---|---|---|
committer | Profpatsch <mail@profpatsch.de> | 2021-05-17T23·00+0000 |
commit | 72924facaebb9cf37d9cfd1da43335d5fe51fb6e (patch) | |
tree | 9c805f378aa315ef7500d9c2308ec50148ddf9cf | |
parent | 8dfc7a9f9d8943cfc56721c09ae9c04090a8f500 (diff) |
fix(nix/writers/rust): remove args override r/2594
We can be closed world, so let’s restrict the arguments to the subset we need for now. The existing override was wrong, in that `// args` would use the arguments we already added, again. So instead of deliberating about how to make this work right in all cases, we don’t need it, we trim it. Change-Id: I6443a0808b8bfd5e4db939b669c6afc741954db8 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3057 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
-rw-r--r-- | nix/writers/default.nix | 14 | ||||
-rw-r--r-- | users/Profpatsch/netencode/default.nix | 2 |
2 files changed, 5 insertions, 11 deletions
diff --git a/nix/writers/default.nix b/nix/writers/default.nix index f43761865fc2..b0795ab2c886 100644 --- a/nix/writers/default.nix +++ b/nix/writers/default.nix @@ -9,9 +9,7 @@ let bins.s6-ln "-s" path "$out" ]; - # Build a rust executable. - # Takes all arguments that `pkgs.buildRustCrate` accepts. - # Rather leaky abstraction. + # Build a rust executable, $out is the executable. rustSimple = args@{name, ...}: src: linkTo name "${rustSimpleBin args src}/bin/${name}"; @@ -20,8 +18,7 @@ let name, dependencies ? [], doCheck ? true, - ... - }@args: src: + }: src: (if doCheck then testRustSimple else pkgs.lib.id) (pkgs.buildRustCrate ({ pname = name; @@ -37,7 +34,7 @@ let cp "$srcPath" $out/src/bin/${name}.rs find $out ''; - } // args)); + })); # Build a rust library, that can be used as dependency to `rustSimple`. # Wrapper around `pkgs.buildRustCrate`, takes all its arguments. @@ -45,8 +42,7 @@ let name, dependencies ? [], doCheck ? true, - ... - }@args: src: + }: src: (if doCheck then testRustSimple else pkgs.lib.id) (pkgs.buildRustCrate ({ pname = name; @@ -61,7 +57,7 @@ let cp "$srcPath" $out/src/lib.rs find $out ''; - } // args)); + })); /* Takes a `buildRustCrate` derivation as an input, * builds it with `{ buildTests = true; }` and runs diff --git a/users/Profpatsch/netencode/default.nix b/users/Profpatsch/netencode/default.nix index 6662e622e1bf..db892cc9de8d 100644 --- a/users/Profpatsch/netencode/default.nix +++ b/users/Profpatsch/netencode/default.nix @@ -7,8 +7,6 @@ let depot.third_party.rust-crates.nom depot.users.Profpatsch.execline.exec-helpers ]; - release = false; - verbose = true; } (builtins.readFile ./netencode.rs); gen = import ./gen.nix { inherit lib; }; |