You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It appears that everything in this project is using the package name Sirupsen/logrus instead of sirupsen/logrus as is suggested by the readme:
Seeing weird case-sensitive problems? Unfortunately, the author failed to realize the consequences of renaming to lower-case. Due to the Go package environment, this caused issues. Regretfully, there's no turning back now. Everything using logrus will need to use the lower-case: github.com/sirupsen/logrus. Any package that isn't, should be changed.
moby/moby is fixing it (i.e. using sirupsen lowercase) and as it is a dependency of cli (as docker/docker for now), we are also going to handle that in this repository. This issue is there to track that move.
@vdemeester ok I've just encountered that issue... as I wanted to upgrade github.com/docker/docker hash to the one of the merge of moby/moby#34590 & continue work on moby/moby#32816 fix so this is blocking me ATM.
I've check and looks like that only notary is waiting for notaryproject/notary#1211 to get merged then it will be ok no? All the other dependencies have been fixed, so close to a cli patch 😉.
Related to moby/moby#33641, from initial issue on
moby/moby
It appears that everything in this project is using the package name
Sirupsen/logrus
instead ofsirupsen/logrus
as is suggested by the readme:moby/moby
is fixing it (i.e. usingsirupsen
lowercase) and as it is a dependency ofcli
(asdocker/docker
for now), we are also going to handle that in this repository. This issue is there to track that move.The text was updated successfully, but these errors were encountered: