guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH


From: Maxim Cournoyer
Subject: Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable.
Date: Sun, 25 Oct 2020 17:37:18 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)

Hello Pierre,

Pierre Neidhardt <mail@ambrevar.xyz> writes:

> Maxim Cournoyer <maxim.cournoyer@gmail.com> writes:
>
>> +                    (clean-geiser-guile-load-path
>> +                     (seq-remove (lambda (x)
>> +                                   (string-match "/guix" x))
>> +                                 geiser-guile-load-path)))
>
> I suggest making "/guix" a let-bound variable and add a comment
> explaining that channel maintainers need to change it.

I'll try the suggestions that Miguel proposed in another reply in this
thread, that should remove the need for this hard-coded value.

> That said, why do we need to clean the load path?  Can't we just leave
> the other paths?  What if the user actually wants the other elements?

I haven't investigated why exactly, but Guile's would get confused when
multiple Guixes were in geiser-guile-load-path and attempt to recompile
everything.

Maxim



reply via email to

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