Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

unexpected module path "golang.ord/x/lint" on make all #132

Closed
stianlagstad opened this issue May 8, 2019 · 3 comments
Closed

unexpected module path "golang.ord/x/lint" on make all #132

stianlagstad opened this issue May 8, 2019 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@stianlagstad
Copy link

stianlagstad commented May 8, 2019

Hi,

I'm believe I'm running into this issue when running make all:

➜  uname -a
Linux stian-Precision-5530 4.18.0-18-generic #19~18.04.1-Ubuntu SMP Fri Apr 5 10:22:13 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
➜  go version
go version go1.12.4 linux/amd64
➜  pwd
/home/stian/go/src/github.com/Azure/azure-service-bus-go
➜  make all
▶ running gofmt…
▶ building golint…
go: finding github.com/golang/lint/golint latest
go: finding github.com/golang/lint latest
go: github.com/golang/lint@v0.0.0-20190409202823-959b441ac422: parsing go.mod: unexpected module path "golang.org/x/lint"
go: error loading module requirements
Makefile:28: recipe for target '/home/stian/go/bin/golint' failed
make: *** [/home/stian/go/bin/golint] Error 1

Is there something I can do to bypass that?

I'm new to golang and to this library, so please don't hesitate to close this issue and tell me to debug further myself.

@stianlagstad
Copy link
Author

I of course figured out a way to solve it a couple of minutes after creating the issue. After doing go get -u golang.org/x/lint/golint to install golint on my system, make all got past that error and succeeded.

I do see golint mentioned in the contributing page, but could it also be worth modifying the Makefile to output a better error for this specific case, and/or add a How to build this project section in the readme?

@devigned devigned added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels May 21, 2019
@devigned
Copy link
Member

Noted and will address time willing :).

@jhendrixMSFT
Copy link
Member

This has been fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants