guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] doc: Fix 'geiser-guile-load-path' example.


From: Ludovic Courtès
Subject: Re: [PATCH] doc: Fix 'geiser-guile-load-path' example.
Date: Tue, 17 Nov 2015 17:16:57 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Alex Kost <address@hidden> skribis:

> Ludovic Courtès (2015-11-15 20:41 +0300) wrote:
>
>> Alex Kost <address@hidden> skribis:
>>
>>> (add-to-list 'geiser-guile-load-path 'foo) errors if "geiser-guile.el"
>>> is not loaded yet, so I think it's better to put it inside
>>> 'with-eval-after-load', WDYT?
>>
>> Sounds good.  I guess that’s because (require 'guix-init) doesn’t
>> require 'geiser?
>
> "guix-init.el" shouldn't require geiser.  IMO it should load as less
> code as possible, because it will increase the Emacs start time.
>
> In general I think requiring/loading packages is evil (as it slows down
> starting Emacs), so I avoid 'require'-s when possible in my init file.

Agreed, I was just making sure I understood the issue at hand.

The patch you sent looks good to me then.

Thanks!

Ludo’.



reply via email to

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