[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’.