[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
- [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals, Maxim Cournoyer, 2020/10/25
- Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals, Pierre Neidhardt, 2020/10/25
- [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/25
- Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Pierre Neidhardt, 2020/10/25
- Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable.,
Maxim Cournoyer <=
- Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Miguel Ángel Arruga Vivas, 2020/10/25
- Re: [PATCH] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/26
- [PATCH v2] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/26
- Re: [PATCH v2] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Pierre Neidhardt, 2020/10/26
- Re: [PATCH v2] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Miguel Ángel Arruga Vivas, 2020/10/26
- Re: [PATCH v2] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/27
- Re: [PATCH v2] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Miguel Ángel Arruga Vivas, 2020/10/27
- [PATCH v3] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/27
- Re: [PATCH v3] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Maxim Cournoyer, 2020/10/31
- Re: [PATCH v3] .dir-locals.el: Automatically set the GEISER-GUILE-LOAD-PATH variable., Miguel Ángel Arruga Vivas, 2020/10/31