guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#31952] [PATCH] gnu: git: Update to 2.18.0.


From: Ludovic Courtès
Subject: [bug#31952] [PATCH] gnu: git: Update to 2.18.0.
Date: Sat, 23 Jun 2018 23:59:20 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Hi Marius!

Marius Bakke <address@hidden> skribis:

> * gnu/packages/version-control.scm (git): Update to 2.18.0.
> [native-inputs]: Add BASH.
> [arguments]: In #:make-flags, define SHELL_PATH and TEST_SHELL_PATH.
> Add #:disallowed-references.  Drop /bin/sh substitution from Makefile.  Add
> phase to prevent BASH from ending up in PATH.  Delete 'patch-shebangs' phase.

[...]

> +        ;; FIXME: This phase picks up the wrong bash when patching shebangs.
> +        (delete 'patch-shebangs)

Do the installed scripts still have the right shebang in spite of this?

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.

WDYT?

Apart from that it LGTM.

Thank you!

Ludo’.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]