diff options
| author | Lucas Hoffmann <luc@artsoftandmore.com> | 2025-11-03 17:05:00 +0100 |
|---|---|---|
| committer | Lucas Hoffmann <luc@artsoftandmore.com> | 2025-11-04 11:47:53 +0100 |
| commit | 48c9505ba112dfadd227bb1cec59125d1be9cf2e (patch) | |
| tree | 37ffb042bd0433b19e0fe307004cd69e6c327f05 /pkgs/build-support/php | |
| parent | gl-gsync-demo: move to by-name, modernize (#450992) (diff) | |
| download | nixpkgs-48c9505ba112dfadd227bb1cec59125d1be9cf2e.tar.gz | |
build-support/php: fix variable substitution in bash
The build environment sets the bash option NOUNSET which fails the
script for any substitution of uninitialized variables. The ${VAR-}
construct circumvents this.
This was tested by building the following derivations:
- phpPackages.box{,.composerVendor}
- phpPackages.cyclonedx-php-composer
- phpPackages.phing{,.composerVendor}
- phpPackages.php-parallel-lint{,.composerVendor}
- phpPackages.castor{,.composerVendor}
- phpPackages.phive{,.composerVendor}
- phpPackages.php-cs-fixer{,.composerVendor}
- phpPackages.phpspy
- phpPackages.composer
- phpPackages.grumphp{,.composerVendor}
- phpPackages.phpinsights{,.composerVendor}
- phpPackages.phpstan{,.composerVendor}
- phpPackages.composer-local-repo-plugin
- phpPackages.phan
- phpPackages.php-codesniffer{,.composerVendor}
- phpPackages.phpmd{,.composerVendor}
- phpPackages.psalm{,.composerVendor}
- psysh{,.composerVendor}
- deployer{,.composerVendor}
Diffstat (limited to 'pkgs/build-support/php')
| -rw-r--r-- | pkgs/build-support/php/builders/v2/hooks/composer-vendor-hook.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/pkgs/build-support/php/builders/v2/hooks/composer-vendor-hook.sh b/pkgs/build-support/php/builders/v2/hooks/composer-vendor-hook.sh index 53503194cfa4..83dda99f33cd 100644 --- a/pkgs/build-support/php/builders/v2/hooks/composer-vendor-hook.sh +++ b/pkgs/build-support/php/builders/v2/hooks/composer-vendor-hook.sh @@ -10,9 +10,9 @@ declare -g composerNoPlugins declare -g composerNoScripts declare -ga composerFlags=() -[[ -n "$composerNoDev" ]] && composerFlags+=(--no-dev) -[[ -n "$composerNoPlugins" ]] && composerFlags+=(--no-plugins) -[[ -n "$composerNoScripts" ]] && composerFlags+=(--no-scripts) +[[ -n "${composerNoDev-}" ]] && composerFlags+=(--no-dev) +[[ -n "${composerNoPlugins-}" ]] && composerFlags+=(--no-plugins) +[[ -n "${composerNoScripts-}" ]] && composerFlags+=(--no-scripts) preConfigureHooks+=(composerVendorConfigureHook) preBuildHooks+=(composerVendorBuildHook) |