Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-18 Thread Miguel Ángel Arruga Vivas
Hello Maxim, Maxim Cournoyer writes: > Hello Miguel, > > Miguel Ángel Arruga Vivas writes: > >> Hi! >> >> Christopher Lemmer Webber writes: >> [...] >>> It sounds like you're already putting together the work to do it. If >>> you don't mind doing it that would save me a lot of time right >>>

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-17 Thread Maxim Cournoyer
Hello Miguel, Miguel Ángel Arruga Vivas writes: > Hi! > > Christopher Lemmer Webber writes: > [...] >> It sounds like you're already putting together the work to do it. If >> you don't mind doing it that would save me a lot of time right >> now... I'm quite swamped! I'd be very grateful! > >

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-16 Thread Christopher Lemmer Webber
Miguel Ángel Arruga Vivas writes: > Hi! > > Christopher Lemmer Webber writes: > [...] >> It sounds like you're already putting together the work to do it. If >> you don't mind doing it that would save me a lot of time right >> now... I'm quite swamped! I'd be very grateful! > > No problem, I ju

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-16 Thread Miguel Ángel Arruga Vivas
Hi! Christopher Lemmer Webber writes: [...] > It sounds like you're already putting together the work to do it. If > you don't mind doing it that would save me a lot of time right > now... I'm quite swamped! I'd be very grateful! No problem, I just didn't want to end up with two reports for th

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-16 Thread Christopher Lemmer Webber
Christopher Lemmer Webber writes: > Miguel Ángel Arruga Vivas writes: > >> I haven't reported it yet, but as you can see I have a reproducer script >> attached. I haven't seen anything in vc-code that points in that >> direction, surely though Emacs people will have a better understanding. >> >>

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-16 Thread Christopher Lemmer Webber
Miguel Ángel Arruga Vivas writes: > Hi Maxim and Christopher, > > Maxim Cournoyer writes: >>[...] >> Christopher Lemmer Webber writes: >> [...] >>> I figured out what was happening! The bug is *technically* in vc-mode. >>> However, nontheless it manifested here... >>> >>> Here's what happened.

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-16 Thread Miguel Ángel Arruga Vivas
Hi Maxim and Christopher, Maxim Cournoyer writes: >[...] > Christopher Lemmer Webber writes: > [...] >> I figured out what was happening! The bug is *technically* in vc-mode. >> However, nontheless it manifested here... >> >> Here's what happened. vc-mode has some various hacks, as you can see

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-15 Thread Maxim Cournoyer
Hello Christopher, CC'ing Ludovic since they had that problem as well, so might be interested in testing your solution. Christopher Lemmer Webber writes: [...] > I figured out what was happening! The bug is *technically* in vc-mode. > However, nontheless it manifested here... > > Here's what

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-14 Thread Christopher Lemmer Webber
Miguel Ángel Arruga Vivas writes: > Hi Christopher, > > First of all I want to say sorry: I've tested this and reviewed the > patch, and this is the second issue that already has caused, so yes, my > tests weren't enough at all. > > Christopher Lemmer Webber writes: > >> Also, I hope this email i

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-05 Thread Christopher Lemmer Webber
Miguel Ángel Arruga Vivas writes: > Hi Christopher, > > First of all I want to say sorry: I've tested this and reviewed the > patch, and this is the second issue that already has caused, so yes, my > tests weren't enough at all. > > Christopher Lemmer Webber writes: > >> Also, I hope this email i

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-05 Thread Maxim Cournoyer
Hi Christopher, Christopher Lemmer Webber writes: [...] > I'm a bit unsure what the implications fully are with this change, but > here was my user experience: > > - Did a pull using magit to guix > - Suddenly every file I open in Guix is prompting me if I want to >enable these dir-locals

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-05 Thread Miguel Ángel Arruga Vivas
Hi Christopher, First of all I want to say sorry: I've tested this and reviewed the patch, and this is the second issue that already has caused, so yes, my tests weren't enough at all. Christopher Lemmer Webber writes: > Also, I hope this email isn't interpreted as being dismissive or negative

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-04 Thread Christopher Lemmer Webber
Also, I hope this email isn't interpreted as being dismissive or negative about what looks like it's probably a real usability improvement for hacking on Guix! I just thought my experience indicated maybe there are additional considerations to address. Christopher Lemmer Webber writes: > Maxim C

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-11-04 Thread Christopher Lemmer Webber
Maxim Cournoyer writes: > Hello Guix! > > I've been experimenting with the following modification to the > .dir-locals file shipped with Guix, that allows setting per-buffer > variables within Emacs when visiting a file in the same directory (or in > one of its sub-directories). > > This makes lif

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-10-26 Thread Pierre Neidhardt
zimoun writes: > Does that mean that the term “guix” should be in the absolute path name, > i.e., “guix/my-worktree-name”? Or does it mean that the term “guix” > should start the worktree, i.e., “guix-my-worktree-name”? The latter, but this is no longer needed in v2. -- Pierre Neidhardt https

Re: [PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-10-26 Thread zimoun
Hi Maxim, Thank you for working on this! On Sun, 25 Oct 2020 at 14:08, Maxim Cournoyer wrote: > The only requirement for it to work reliably is that any Guix checkout > or worktree name should start with "guix". I have read all the thread and the comments, neither v2. What do you mean by “any

[PATCH] Automatically set `geiser-guile-load-path' from .dir-locals

2020-10-25 Thread Maxim Cournoyer
Hello Guix! I've been experimenting with the following modification to the .dir-locals file shipped with Guix, that allows setting per-buffer variables within Emacs when visiting a file in the same directory (or in one of its sub-directories). This makes life a bit easier, by ensuring that a Geis