GNU bug report logs - #55081
[PATCH v1] Allow Guix root-dir overrides when working via Tramp.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 55081 in the body.
You can then email your comments to 55081 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Brian Cully <bjc <at> spork.org>
To: guix-patches <at> gnu.org
Subject: [PATCH v1] Allow Guix root-dir overrides when working via Tramp.
Date: Sat, 23 Apr 2022 12:17:52 -0400

Add only the local part of the root directory to the guile search
path. This lets you use Tramp to hack on a remote Guix checkout.
---
 .dir-locals.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.dir-locals.el b/.dir-locals.el
index 09e19223d5..3b75989133 100644
--- a/.dir-locals.el
+++ b/.dir-locals.el
@@ -29,7 +29,7 @@
                ;; Hence the following "when", which might otherwise be unnecessary;
                ;; it prevents causing an error when root-dir-unexpanded is nil.
                (when root-dir-unexpanded
-                 (let* ((root-dir (expand-file-name root-dir-unexpanded))
+                 (let* ((root-dir (file-local-name (expand-file-name root-dir-unexpanded)))
                         ;; Workaround for bug https://issues.guix.gnu.org/43818.
                         (root-dir* (directory-file-name root-dir)))
 

base-commit: 8540c2d86ca535ceb188c838cf1885de6f65c94f
-- 
2.35.1

Message #8 received at 55081 <at> debbugs.gnu.org (full text, mbox):

From: Brian Cully <bjc <at> spork.org>
To: 55081 <at> debbugs.gnu.org
Cc: Brian Cully <bjc <at> kublai.com>
Subject: [PATCH v2] Allow Guix root-dir overrides when working via Tramp.
Date: Sat, 21 May 2022 15:10:24 -0400

From: Brian Cully <bjc <at> kublai.com>

Add only the local part of the root directory to the guile search
path. This lets you use Tramp to hack on a remote Guix checkout.
---
I think my last patch was mangled by GNUS, so I'm resending it.

FWIW, I've been using this for months without issue.
 .dir-locals.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.dir-locals.el b/.dir-locals.el
index 09e19223d5..3b75989133 100644
--- a/.dir-locals.el
+++ b/.dir-locals.el
@@ -29,7 +29,7 @@
                ;; Hence the following "when", which might otherwise be unnecessary;
                ;; it prevents causing an error when root-dir-unexpanded is nil.
                (when root-dir-unexpanded
-                 (let* ((root-dir (expand-file-name root-dir-unexpanded))
+                 (let* ((root-dir (file-local-name (expand-file-name root-dir-unexpanded)))
                         ;; Workaround for bug https://issues.guix.gnu.org/43818.
                         (root-dir* (directory-file-name root-dir)))
 
-- 
2.36.1

Message #11 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Leo Famulari <leo <at> famulari.name>
To: Brian Cully via Guix-patches via <guix-patches <at> gnu.org>
Cc: 55081-done <at> debbugs.gnu.org, Brian Cully <bjc <at> kublai.com>
Subject: Re: [bug#55081] [PATCH v2] Allow Guix root-dir overrides when
 working via Tramp.
Date: Sun, 5 Mar 2023 14:06:20 -0500

On Sat, May 21, 2022 at 03:10:24PM -0400, Brian Cully via Guix-patches via wrote:
> From: Brian Cully <bjc <at> kublai.com>
> 
> Add only the local part of the root directory to the guile search
> path. This lets you use Tramp to hack on a remote Guix checkout.
> ---
> I think my last patch was mangled by GNUS, so I'm resending it.
> 
> FWIW, I've been using this for months without issue.

Okay, pushed as cedf97ed6ee4eba8c39bfe6cc0efe33fcb977ccf

Thanks!

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.