search

emacs-pe / docker-tramp.el

TRAMP integration for docker containers
340 stars 25 forks source link

Support completion in multi-hop paths #36

Open cl-user opened 2 years ago

cl-user commented 2 years ago

Fixes emacs-pe/docker-tramp.el#17.

olessavluk commented 2 years ago

Nice, exactly what I wanted to achieve! Just tried running locally - completion seems to be working well 🚀

PRESFIL commented 2 years ago

Works perfectly except for one small moment. If the connection to the Docker fails, the docker-tramp--running-containers still continues to generate pairs :smile: :

(("Cannot") ("connect") ("to") ("the") ("Docker") ("daemon") ("at") ("unix" "///var/run/docker.sock.") ("Is") ("the") ("docker") ("daemon") ...)

Perhaps an additional check for a non-zero return code is needed here.

p.s. I don’t know how the Docker daemon didn’t start for me like the first time, after all I have systemd socket activation. Now I can't reproduce this error.

PRESFIL commented 1 year ago

Any updates on this?