[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#68180] [PATCH 1/4] gnu: emacs: Add awk, find, sed and sh to PATH wr
From: |
Maxim Cournoyer |
Subject: |
[bug#68180] [PATCH 1/4] gnu: emacs: Add awk, find, sed and sh to PATH wrapper. |
Date: |
Sun, 31 Dec 2023 16:10:03 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Hi Liliana,
Liliana Marie Prikler <liliana.prikler@gmail.com> writes:
> Am Sonntag, dem 31.12.2023 um 11:59 -0500 schrieb Maxim Cournoyer:
>> Before this change, using Emacs in a pure environment, e.g. 'guix
>> shell --pure emacs', would cause problems such as:
>>
>> jka-compr-insert-file-contents: Uncompression program ‘sh’ not
>> found
>>
>> And other problems were found requiring the other tools. While the
>> above could be patched in place for 'sh', it seems more robust and
>> universally useful to have the commands appear on PATH, should other
>> Elisp modules want to call to these directly as well.
>>
>> * gnu/packages/emacs.scm (emacs-minimal) [arguments] <phases>: Adjust
>> the
>> wrap-emacs-paths phase to wrap additional inputs.
>> [inputs]: Add findutils, gawk and sed.
>> (%emacs-modules): Add (srfi srfi-26).
>>
>> Change-Id: Ifb4fe2fc12ddc9eae387adb3da3f7821fab78e65
>> ---
> We already have a phase to patch in the real path of /bin/sh where it's
> used. This appears to be an odd case that's missed.
I appreciate exactness, but it seems fragile to rely on nobody adding
new references or someone catching them as new Emacs modules get added
or changed :-).
My reasoning was that since Emacs already depends on bash, why not
ensure it'll always be found on PATH, by wrapping instead of
substituting.
Does it make sense?
--
Thanks,
Maxim