[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#43627] [PATCH core-updates]: Add a 'append-separator?' field to the
From: |
Maxim Cournoyer |
Subject: |
[bug#43627] [PATCH core-updates]: Add a 'append-separator?' field to the <search-path-specification> record. |
Date: |
Sun, 08 Nov 2020 00:49:32 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Hi Ludovic,
Ludovic Courtès <ludo@gnu.org> writes:
> Hi Maxim,
>
> Apologies for not following up earlier on this one…
No worries!
> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>
>> Ludovic Courtès <ludo@gnu.org> writes:
>>
>>> Hi,
>>>
>>> Maxim Cournoyer <maxim.cournoyer@gmail.com> skribis:
>>>
>>>> These three commits extend our search-path-specification record with a
>>>> new field, that can be used to produce a trailing separator, which can
>>>> be useful at least with Emacs (I can think of at least another place
>>>> where it could be used: the INFOPATH variable).
>>>>
>>>> It was motivated to allow defining the Emacs search path in a more
>>>> robust way.
>>>
>>> I’m skeptical since we have only one (or two?) use cases. I think we
>>> should weigh the added complexity, both in terms of implementation and
>>> of semantic clarity, compared to reduced complexity elsewhere. It seems
>>> to me that the costs outweigh the benefits here.
>>
>> I too was skeptical at first (which explains why this commit was not
>> submitted for inclusion for 3 years :-)), but recent events surrounding
>> emacs-next and the bump to Emacs 27 have put its value back into light.
>
> Do you know if there are other use cases than Emacs for this?
The three environment variables that I know that make use of it in this
way are EMACSLOADPATH, INFOPATH and MANPATH. There may be others, but
it's hard to search for that.
> I vaguely remember that ‘EMACSLOADPATH’ unusual handling of the empty
> entry is why we didn’t have it in ‘native-search-paths’ initially, and
> why Alex Kost had come up with a different mechanism instead.
I'm not sure what this was about, but if there was an issue with using
EMACSLOADPATH, I think we would have found out by now :-).
Thanks,
Maxim