Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency on wrong-case version of logrus #1

Closed
joemcmahon opened this issue Sep 3, 2018 · 2 comments
Closed

Dependency on wrong-case version of logrus #1

joemcmahon opened this issue Sep 3, 2018 · 2 comments

Comments

@joemcmahon
Copy link

If you've already followed the instructions at github.com/sirupsen/logrus (note lower-case), then this module fails to resolve properly during dep ensure.

(12)  ? attempt github.com/allenluce/logcap with 1 pkgs; 1 versions to try
(13)      try github.com/allenluce/logcap@master
(14)  ✗   case-only variation in dependency on "github.com/Sirupsen/logrus"; "github.com/sirupsen/logrus" already established by:
(14)    (root)
(13)    ← no more versions of github.com/allenluce/logcap to try; begin backtrack
(12)  ← backtrack: no more versions of github.com/spf13/pflag to try
(11)  ← backtrack: no more versions of github.com/inconshreveable/mousetrap to try
(10)  ← backtrack: no more versions of github.com/spf13/cobra to try
(9)	← backtrack: no more versions of golang.org/x/net to try
(8)	← backtrack: no more versions of github.com/mitchellh/go-homedir to try
(7)	← backtrack: popped 5 pkgs from github.com/golang/protobuf
(6)	← backtrack: no more versions of google.golang.org/grpc to try
(5)	← backtrack: no more versions of github.com/sirupsen/logrus to try
(4)	← backtrack: no more versions of github.com/hpcloud/tail to try
(3)	← backtrack: no more versions of github.com/onsi/ginkgo to try
(2)	← backtrack: no more versions of github.com/golang/protobuf to try
(1)	← backtrack: no more versions of github.com/disiqueira/gotree to try
  ✗ solving failed

It would probably be better to change the case of the import in this module too. I'm sure there's a dep workaround, and I will use it -- but fixing this is probably better in the long run.

@allenluce
Copy link
Owner

Sorry I'd missed this! I actually put in the fix before noticing you'd opened an issue & submitted a pull request on it.

@joemcmahon
Copy link
Author

joemcmahon commented Oct 9, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants