about summary refs log tree commit diff
path: root/nix/writers
diff options
context:
space:
mode:
authorProfpatsch <mail@profpatsch.de>2021-04-24T11·32+0200
committerProfpatsch <mail@profpatsch.de>2021-05-17T23·00+0000
commit72924facaebb9cf37d9cfd1da43335d5fe51fb6e (patch)
tree9c805f378aa315ef7500d9c2308ec50148ddf9cf /nix/writers
parent8dfc7a9f9d8943cfc56721c09ae9c04090a8f500 (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>
Diffstat (limited to 'nix/writers')
-rw-r--r--nix/writers/default.nix14
1 files changed, 5 insertions, 9 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