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

Jump to bottom

cmd/go: make -mod=vendor a noop in GOPATH mode #29665

Closed
kardianos opened this issue Jan 10, 2019 · 2 comments
Closed

cmd/go: make -mod=vendor a noop in GOPATH mode #29665

kardianos opened this issue Jan 10, 2019 · 2 comments
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Go1.12

Comments

@kardianos
Copy link
Contributor

@bcmills

Reproduces as of 2018-01-10 at HEAD.

In my build scripts I've added -mod=vendor for a project. This works great when outside of GOPATH. On a different computer that still has the source inside GOPATH, I get an error:
build flag -mod=vendor only valid when using modules

This error seems like a mistake. I just want to tell go to use the vendor folder. If in GOPATH mode, it will already do that. But go mod mode currently requires a flag. Is there a situation where someone would pass in -mod=vendor while in GOPATH mode and it would do something unexpected?

This is a sub-issue of #27227, but perhaps it would be small and simple enough to make it in 1.12.
@cespare
Copy link
Contributor

cespare commented Jan 10, 2019 •
edited

I ran into this as part of looking into #27227 (comment). One thing I noticed is that if you specify this not via the flag but instead using GOFLAGS='-mod=vendor', then it works fine in GOPATH mode.

@julieqiu julieqiu added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. modules labels Jan 10, 2019
@julieqiu julieqiu added this to the Go1.12 milestone Jan 10, 2019
@bcmills
Copy link
Contributor

bcmills commented Jan 11, 2019

Given that you can use GOFLAGS as a workaround — and given how far into the 1.12 freeze we are — I'd rather not add the -mod flag to GOPATH mode.

Sorry, but let's wait and address #27227 holistically.

@bcmills bcmills closed this as completed Jan 11, 2019
@golang golang locked and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge modules NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Go1.12
Development

No branches or pull requests
5 participants
@cespare @kardianos @bcmills @julieqiu @gopherbot