[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#31952: [PATCH] gnu: git: Update to 2.18.0.
From: |
Marius Bakke |
Subject: |
bug#31952: [PATCH] gnu: git: Update to 2.18.0. |
Date: |
Sun, 24 Jun 2018 22:38:17 +0200 |
User-agent: |
Notmuch/0.27 (https://notmuchmail.org) Emacs/26.1 (x86_64-pc-linux-gnu) |
address@hidden (Ludovic Courtès) writes:
>>> Removing this phase altogether sounds a bit risky. Another option would
>>> have been to replace it with one that moves the “right” Bash to the
>>> front of PATH and then calls the original ‘patch-shebangs’ phase.
>>
>> The problem is that 'patch-shebangs' does not use PATH, but instead
>> iterates over inputs directly. It's supposed to prefer 'inputs' to
>> 'native-inputs' (according to a comment), yet in this case it picks the
>> native "full" bash rather than bash-minimal.
>
> OK.
>
>> If you read closely, you'll notice that 'bash-for-tests' is not in PATH
>> at all. 'patch-source-shebangs' and other things that use (which "sh")
>> works okay due to that.
>
> It may be that moving “bash-for-tests” to ‘inputs’ actually solves the
> problem (and we don’t have to worry about cross-compilation since “make
> check” does nothing when cross-compiling) and we can keep the
> ‘patch-shebangs’ phase.
Moving it to 'inputs' worked, though I'm not sure why!
I pushed the patch with that change, thanks for your feedback :-)
signature.asc
Description: PGP signature