kinvolk/lokomotive

lokoctl go requirement not properly defined

smokes2345 opened this issue · 2 comments

smokes2345 commented

Description

Installer requirement github.com/Sirupsen/logrus not defined correctly

Impact

Cannot install lokoctl using "go get..." method

Environment and steps to reproduce

  1. Set-up: brew install golang
  2. Task: ???
  3. Action(s): go get github.com/kinvolk/lokomotive/cmd/lokoctl
  4. Error:
➜  workspace git:(master) ✗ go get github.com/kinvolk/lokomotive/cmd/lokoctl
go get: github.com/Sirupsen/logrus@none updating to
        github.com/Sirupsen/logrus@v1.8.1: parsing go.mod:
        module declares its path as: github.com/sirupsen/logrus
                but was required as: github.com/Sirupsen/logrus

Expected behavior

lokoctl installed

Additional information

Please add any information here that does not fit the above format.

invidian commented

Hmm, I guess this is likely because of multiple replace statements in go.mod file we have, which are not respected by go get or go install.

TheApeMachine commented

Just lowercase Sirupsen right? This logging package has been a problem ever since he lowercased the letter :p

  • 😄1