[Top][All Lists]

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

Re: master 152c7322c1: * lisp/net/tramp-docker.el: New file.

From: Michael Albinus
Subject: Re: master 152c7322c1: * lisp/net/tramp-docker.el: New file.
Date: Mon, 03 Oct 2022 19:37:33 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux)

Brian Cully <bjc@spork.org> writes:

Hi Brian,

> Indeed. I hadn't considered this use-case (even though I also have
> both installed, though I only use Tramp with Podman containers for
> development purposes).
> As the package is really not much more than the method and variable
> customization options, it may be best to just have duplicates for
> those (eg, both `tramp-docker-program' and
> `tramp-podman-program'). The completion code could be shared, but I'm
> unsure how higher-level code organization would work. My first pass
> would be to have a single `tramp-docker--completion-function' which is
> used for both the `tramp-podman-method' as well as
> `tramp-docker-method'. My rationale being that Podman explicitly
> copies the Docker command line so that at can be used as a drop-in
> replacement, so the completion function would more naturally be
> associated with Docker than Podman.

OK. Would you like to provide a patch? Including a ChangeLog-like commit

> -bjc

Best regards, Michael.

reply via email to

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