Empty lists of images and containers

Silex / docker.el

Manage docker from Emacs.

745 stars 74 forks source link

Empty lists of images and containers #195

Closed zeeros closed 2 years ago

zeeros commented 2 years ago

I have this problem with both melpa and melpa-stable versions. Containers and images are not displayed.

Also, if I try to pull a new image nothing happens.

Silex commented 2 years ago

Can you show me what is displayed in the *Messages* buffer (C-h e) when you try to display the images list?

zeeros commented 2 years ago

Nothing is printed out in *Messages*

Silex commented 2 years ago

What is your emacs version? Can you try with emacs -Q?

zeeros commented 2 years ago

Emacs 27.2

Silex commented 2 years ago

This is really weird.

What does happen when you do: M-x eval-expression RET (aio-wait-for (docker-image-entries)) RET ?

zeeros commented 2 years ago

Symbol’s function definition is void: transient-default-value

Silex commented 2 years ago

Seems like you get hit by https://github.com/magit/transient/issues/190

But that does only happen with MELPA-stable, and you said you also tested with unstable.

Can you try with unstable again and make sure transient uses unstable?

zeeros commented 2 years ago

With transient-20220314.1605 both images and containers are correctly displayed

theottm commented 2 years ago

Encountered the same problem. Also nothing in *Messages*. Maybe it would be nice to catch this error ? For people using use-package and straight you can use this to get the latest version of transient:

(use-package transient
  :straight '(transient
              :type git
              :host github
              :repo "magit/transient"
              :commit "2e4426fe8161893382f09b3f4635e152ee02488e"
              ))

Of course replace commit with appropriate commit hash.

sandwoodK commented 11 months ago

Fall on same issue. It may discourage people with very few knowledge of emacs / package of using this. Agree that if possible some trace at least in Messages should point out to something.

Silex commented 11 months ago

Should be fixed by https://github.com/Silex/docker.el/commit/21d0039e0dd5c9c1abc19697cf46874807ae437e, not sure why I missed this simple fix.

Silex commented 11 months ago

Actually no I also need to publish a tag. Ok well time for a release 😅

Silex commented 11 months ago

Release is done 🥳 this should not happen anymore.