Re: [PATCHES] Do not use special site-start.el on GuixSD

[Ludovic Courtès]

Alex Kost <address@hidden> skribis:

> Alex Kost (2016-02-10 11:48 +0300) wrote:
>
>> Ludovic Courtès (2016-02-06 15:41 +0300) wrote:
>>
>>> Alex Kost <address@hidden> skribis:
>>>
>>>> [PATCH 3/4] system: Fix EMACSLOADPATH.
>>>>
>>>>   This is an easy fix, however I've just thought that we can get rid of
>>>>   EMACSLOADPATH completely.  Since now our Emacs can load packages from
>>>>   guix directories ("~/.guix-profile" and "/run/current-system/profile")
>>>>   there is no need to use "/etc/emacs/site-start.el" anymore, because Emacs
>>>>   will load "guix-autoloads.el" and "geiser-autoloads.el" from a system
>>>>   profile.
>>>>
>>>>   So I think this patch is OK as a current fix, but eventually we should
>>>>   remove "guix-init.el" and "/etc/emacs/site-start.el".  I'll send a
>>>>   patch for it later.
>>>
>>> Good, this is better.
>>
>> OK, so I'm sending the following patches for this change:
>>
>> [PATCH 1/3] emacs: Autoload hooks instead of putting them in 'guix-init'.
>   This was committed as 19a9c6f.
>
>> [PATCH 2/3] system: Do not create "site-start.el".
>> [PATCH 3/3] emacs: Deprecate "guix-init.el".
>>
>> However, they should be commited after updating guix-devel snapshot,
>> because with the current source, Emacs finds packages only in a user
>> profile, but not in the system one (it was introduced in commit
>> 004ea62).
>
> Since guix-devel was updated (commit 092dd65), I think it's the time for
> this change.  The patches can be found at ¹ and ², but I reattach them
> to this message as well.  Also I attach another patch to clarify how to
> set up "guix.el".
>
> ¹ http://lists.gnu.org/archive/html/guix-devel/2016-02/msg00362.html
> ² http://lists.gnu.org/archive/html/guix-devel/2016-02/msg00363.html

Sure!

Thanks for keeping it in mind.

Ludo’.