diff options
| author | Lin Yinfeng <lin.yinfeng@outlook.com> | 2022-04-14 11:47:59 +0800 |
|---|---|---|
| committer | Lin Yinfeng <lin.yinfeng@outlook.com> | 2022-04-15 10:35:11 +0800 |
| commit | bedc267a787ee68b4c02f168414f67885907f7b0 (patch) | |
| tree | a84f3eb3bdea255cbd652eb804b6fc7596e08a71 /pkgs/development/python-modules/rangehttpserver | |
| parent | 831a344b8bdb9bfe78aed47e51320b0ef699d198 (diff) | |
autoPatchelfHook: fix precise dependency ignorance
This commit fixes precise dependency ignorance by converting the
environment variable `autoPatchelfIgnoreMissingDeps` into a bash array
`ignoreMissingDepsArray`, passing `"${ignoreMissingDepsArray[@]}"`
instead of `"${autoPatchelfIgnoreMissingDeps[@]}"` to the python
script.
The original implementation does not work when
`autoPatchelfIgnoreMissingDeps` contains multiple dependency names.
Because it mistakenly passes `"${autoPatchelfIgnoreMissingDeps[@]}"`
to the python script. According to the Nix manual
(https://nixos.org/manual/nix/stable/expressions/derivations.html),
lists of strings are concatenated into whitespace-separated strings,
then passed to the builder as environment variables. So, if
`autoPatchelfIgnoreMissingDeps = [ "dep1" "dep2" "dep3" ]`,
`"${autoPatchelfIgnoreMissingDeps[@]}"` will be expanded to a single
argument `"dep1 dep2 dep3"`, which is not the intended behavior,
because the python script takes the long argument as a dependency
name.
With this commit, `"${ignoreMissingDepsArray[@]}"` will be expanded to
three arguments `"dep1" "dep2" "dep3"` arguments as expected, fixing
the issue.
Diffstat (limited to 'pkgs/development/python-modules/rangehttpserver')
0 files changed, 0 insertions, 0 deletions
