Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

lokoctl go requirement not properly defined #1538

Open
smokes2345 opened this issue Jul 22, 2021 · 2 comments
Open

lokoctl go requirement not properly defined #1538

smokes2345 opened this issue Jul 22, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@smokes2345
Copy link

smokes2345 commented Jul 22, 2021

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 invidian changed the title lokctl go requirement not properly defined lokoctl go requirement not properly defined Aug 9, 2021
@invidian invidian added the bug Something isn't working label Aug 9, 2021
@invidian
Copy link
Member

invidian commented Aug 9, 2021

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
Copy link

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants