diff options
author | Alyssa Ross <hi@alyssa.is> | 2018-10-29 21:39:51 +0000 |
---|---|---|
committer | zimbatm <zimbatm@zimbatm.com> | 2018-10-29 22:39:51 +0100 |
commit | 69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a (patch) | |
tree | 403e2a501a86ec4acee69b5ab74269d6ba9c2f70 /pkgs/development/ruby-modules/bundled-common | |
parent | 3947b88389d8d9cc938035b21eb1ee9d1f23b6a6 (diff) | |
download | nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar.gz nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar.bz2 nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar.lz nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar.xz nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.tar.zst nixpkgs-69dcb1a2c0c5596fbbbe7fa50d4f8277c6f1ea6a.zip |
bundlerApp: take buildInputs (#45435)
It would be reasonable to have a Ruby program that depends on some other program being in the PATH. In this case, the obvious thing to do would be something like this: bundlerApp { # ... buildInputs = [ makeWrapper ]; postBuild = '' wrapProgram "$out/bin/foo" \ --prefix PATH : ${lib.makeBinPath [ dep ]} ''; } However, this doesn't work, because even though it just forwards most of its arguments to `runCommand`, `bundlerApp` won't take a `buildInputs` parameter. It doesn't even specify its own `buildInputs`, which means that the `scripts` parameter to `bundlerApp` (which depends on `makeWrapper`) is completely broken, and, as far as I can tell, has been since its inception. I've added a `makeWrapper` build input if the scripts parameter is present to fix this. I've added a `buildInputs` option to `bundlerApp`. It's also passed through to bundled-common because `postBuild` scripts are run there as well. This actually means that in this example we'd end up going through two layers of wrappers (one from `bundlerApp` and one from bundled-common), but that has always been the case and isn't likely to break anything. That oddity does suggest that it might be prudent to not forward `postBuild` to bundled-common (or to at least use a different option) though... FWIW, as far as I can tell no package in nixpkgs uses either the `scripts` or `postBuild` options to `bundlerApp`.
Diffstat (limited to 'pkgs/development/ruby-modules/bundled-common')
-rw-r--r-- | pkgs/development/ruby-modules/bundled-common/default.nix | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/pkgs/development/ruby-modules/bundled-common/default.nix b/pkgs/development/ruby-modules/bundled-common/default.nix index f902ca58c38..415457b86e1 100644 --- a/pkgs/development/ruby-modules/bundled-common/default.nix +++ b/pkgs/development/ruby-modules/bundled-common/default.nix @@ -19,6 +19,7 @@ , meta ? {} , groups ? ["default"] , ignoreCollisions ? false +, buildInputs ? [] , ... }@args: @@ -96,7 +97,7 @@ let envPaths = lib.attrValues gems ++ lib.optional (!hasBundler) bundler; basicEnv = buildEnv { - inherit ignoreCollisions; + inherit buildInputs ignoreCollisions; name = name'; |