tramp-devel
[Top][All Lists]
Advanced

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

Re: tramp (2.6.1 HEAD/3ff676c2f98cb6c47fecb37f31a589a910dd3876); docker-


From: Michael Albinus
Subject: Re: tramp (2.6.1 HEAD/3ff676c2f98cb6c47fecb37f31a589a910dd3876); docker-container ssh multi-hop support
Date: Mon, 31 Jul 2023 20:50:04 +0200
User-agent: Gnus/5.13 (Gnus v5.13)

Gene Goykhman <gene@indigo1.com> writes:

Hi Gene & Kristofer,

> I've made some progress and have published a gist with my current
> approach to multi-hop Docker container completion in TRAMP. I'd
> appreciate comments and suggestions, and whether this is a reasonable
> direction to take or if I'm missing something important.
>
>   https://gist.github.com/genegoykhman/09868acd369b3fae866be6446997de9f
>
> I've taken an approach similar to the one Kristofer describes ...
>
>> A very simple change I tried when previously trying to get multihop
>> completion working
>> with another backend was that if I changed the variable
>> `tramp-compat-temporary-file-directory'
>> for the function `temporary-file-directory' in the completion
>> functions (like `tramp-container--completion-function'), then at least
>
> I'm advising :before tramp-completion-handle-file-name-all-completions
> and parsing out the required remote directory from which we need to
> call the docker program to get remote completions. Then, I'm adding a
> custom completion function my-tramp-container-completion-function that
> takes that directory (if it has been parsed) and sets it as the
> default-directory.
>
>> Another issue that I found, which maybe is unavoidable, is that some
>> of the completion functions in `tramp-container' requires that the
>> container
>> program is installed on the local host, but sometimes I only have
>
> I /think/ the approach I've taken only requires that the container program be 
> available on the remote host on which we are trying to get the completion 
> list.
>
> Feedback appreciated!

Thanks for all your work! The idea is nice. I'll check next days, and
send you feedback.

Unfortunately, I have first to fix the Tramp ELPA package. People have
reported, that it doesn't install any longer with the just-released
Emacs 29.1 ...

Best regards, Michael.



reply via email to

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