Closed mwolson closed 7 years ago
Hi sorry for the delay, I've created the tag so I think should be matter of time available in melpa stable
Works great, thanks!
On this topic, I'd like to add the package to NonGNU ELPA. The only thing necessary for that would be to update the Version attribute in the main file. Would there be any interest in that?
@phikal I'm so sorry for the late response. I've bumped v0.1.1, please let me know if there is anything else I can do to help you.
That should be it, but if you don't mind I have a few questions about the code:
docker-tramp-docker-executable
autoloaded?docker-tramp--running-containers
with a more stable implementation like
(with-temp-buffer
(apply #'call-process docker-tramp-docker-executable nil t nil
(append docker-tramp-docker-options
(list "ps" "--format" "{{.ID}}\036{{.Names}}")))
(let (containers)
(while (search-backward-regexp "^\\(.+\\)\036\\(.+\\)$" nil t)
(push (cons (match-string 1) (match-string 2))
containers))
containers))
I am not a docker expert, so you should double check if this is ok.
Also I should point out the NonGNU ELPA rules for admission. From what I see everything is ok, but I just want to make sure you know about it.
It would be great to have this package on MELPA Stable for easier installing.