Re: branch master updated: gnu: emacs: Add TREE_SITTER_GRAMMAR_PATH support.

["Ludovic Courtès" <ludo@gnu.org>]

Hi,

Andrew Tropin <andrew@trop.in> skribis:

> On 2023-02-11 16:35, Andrew Tropin wrote:

[...]

>>> The change to guix-emacs.el is pretty impactful here since that affects
>>> all the emacs packages including emacs-minimal I think.
>>>
>>> If I've understood your comment correctly, that does seem pretty
>>> wasteful since those rebuilds because of the changed emacs-minimal won't
>>> have any affect since it's version 28 and not 29, right? I think
>>> applying this change to just emacs-next would have been possible.
>>
>> It could be applied to emacs-next only and we could move it to emacs
>> later when it updated to 29, but I checked guix refresh --list-dependent
>> emacs and it shown less than 100 packages so I decided to apply it
>> straight to emacs, but I forgot that emacs-minimal is inherited from
>> emacs and I guess you are right and it will affect all the emacs
>> packages.  Anything we would like to do about it right now?
>
> Let's keep it as it is and I'll store it in mind for the next time.

Right.

> There is an idea to update guix refresh --list-dependent to handle the
> case with inherited packages as well.  WDYT?

Unfortunately, it’s not possible because inheritance info isn’t
available at run time.

However, peer review and <https://qa.guix.gnu.org> should be able to
catch it.  No big deal, but next time we’ll know.

Thanks,
Ludo’.