Merge pull request #222136 from andrewhamon/allow-substitutions-for-write-shell-application

This commit is contained in:
Bernardo Meurer 2023-04-03 08:19:56 -07:00 committed by GitHub
commit e835abc046
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -132,13 +132,12 @@ rec {
, destination ? "" # relative path appended to $out eg "/bin/foo" , destination ? "" # relative path appended to $out eg "/bin/foo"
, checkPhase ? "" # syntax checks, e.g. for scripts , checkPhase ? "" # syntax checks, e.g. for scripts
, meta ? { } , meta ? { }
, allowSubstitutes ? false
, preferLocalBuild ? true
}: }:
runCommand name runCommand name
{ inherit text executable checkPhase meta; { inherit text executable checkPhase meta allowSubstitutes preferLocalBuild;
passAsFile = [ "text" ]; passAsFile = [ "text" ];
# Pointless to do this on a remote machine.
preferLocalBuild = true;
allowSubstitutes = false;
} }
'' ''
target=$out${lib.escapeShellArg destination} target=$out${lib.escapeShellArg destination}
@ -324,6 +323,8 @@ rec {
inherit name; inherit name;
executable = true; executable = true;
destination = "/bin/${name}"; destination = "/bin/${name}";
allowSubstitutes = true;
preferLocalBuild = false;
text = '' text = ''
#!${runtimeShell} #!${runtimeShell}
set -o errexit set -o errexit