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

2015-11-18 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2015-11-18 00:40 +0300) wrote: [...] >> I have: >> >> (require 'geiser-install) >> (require 'geiser) > > You don't need (require 'geiser) as (require 'geiser-install) loads it > anyway (look at "M-x find-library geiser-install"). > > But after this, 'ge

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

2015-11-18 Thread Alex Kost
Ludovic Courtès (2015-11-18 00:40 +0300) wrote: > Alex Kost skribis: > >> OK. Actually I thought that you use 'with-eval-after-load' or >> 'eval-after-load' or 'use-package', and you just didn't mention it in >> the manual because it's a common Emacs thing. But it looks like you use >> a straig

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

2015-11-17 Thread Ludovic Courtès
Alex Kost skribis: > OK. Actually I thought that you use 'with-eval-after-load' or > 'eval-after-load' or 'use-package', and you just didn't mention it in > the manual because it's a common Emacs thing. But it looks like you use > a straightforward (add-to-list ...), which can work only if > "g

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

2015-11-17 Thread Alex Kost
Ludovic Courtès (2015-11-17 19:16 +0300) wrote: > Alex Kost skribis: > >> Ludovic Courtès (2015-11-15 20:41 +0300) wrote: >> >>> Alex Kost 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

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

2015-11-17 Thread Ludovic Courtès
Alex Kost skribis: > Ludovic Courtès (2015-11-15 20:41 +0300) wrote: > >> Alex Kost 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

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

2015-11-17 Thread Alex Kost
Ludovic Courtès (2015-11-15 20:41 +0300) wrote: > Alex Kost 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-i

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

2015-11-16 Thread Ludovic Courtès
Alex Kost 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? Ludo’.

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

2015-11-15 Thread Alex Kost
at, 14 Nov 2015 21:13:07 +0300 Subject: [PATCH] doc: Fix 'geiser-guile-load-path' example. Reported by marusich on #guix. * doc/contributing.texi (The Perfect Setup): Wrap modifying 'geiser-guile-load-path' into 'with-eval-after-load'. --- doc/contributing.texi | 3 +