[Top][All Lists]

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

Re: tramp ( nil/nil); Add Guix System remote path

From: Michael Albinus
Subject: Re: tramp ( nil/nil); Add Guix System remote path
Date: Sun, 10 Oct 2021 09:03:55 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux)

Vivien Kraus <> writes:

> Hello Michael,

Hi Vivien,

> Is it not possible to just test for the existence of the /run/current-
> system/profile/bin (and .../sbin if needed) directory? I’m not familiar
> with the tramp code base, why should tramp even know it is connecting
> to a guix system server? I thought it would just set the PATH to all
> possible default candidates and let the remote shell decide. Is it not
> the case?

Tramp checks for all candidates of tramp-remote-path whether they exist
on the remote host, and filters the non-existing out. This is due to
security reasons.

So yes, it would be feasible just to add these two directories to
tramp-remote-path. However, if I do it in the Tramp codebase
unconditionally, this check will be performed for all users. I suspect
most of them don't connect a remote guix system; this additional check
would be a performance penalty for them during the handshake. And with
the same argument other users could request to add further directories
to tramp-remote-path initial value.

Wouldn't it be much simpler, if you configure tramp-remote-path on your
own? After all, it is a user option, intended for this.

--8<---------------cut here---------------start------------->8---
(add-to-list 'tramp-remote-path "/run/current-system/profile/bin" t)
(add-to-list 'tramp-remote-path "/run/current-system/profile/sbin" t)
--8<---------------cut here---------------end--------------->8---

> Vivien

Best regards, Michael.

reply via email to

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