-
Notifications
You must be signed in to change notification settings - Fork 17.7k
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
go get -v broken after upgrade go1.13.0 #34058
Labels
FrozenDueToAge
modules
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
Comments
Well, after investigation and the codebase is cmd/go/internal/modget/get.go:346 as the following: if !strings.Contains(path, "...") {
pkgPath := modload.DirImportPath(filepath.FromSlash(path))
if pkgs := modload.TargetPackages(pkgPath); len(pkgs) == 0 {
abs, err := filepath.Abs(path)
if err != nil {
abs = path
}
base.Errorf("go get %s: path %s is not a package in module rooted at %s", arg, abs, modload.ModRoot())
continue
}
}
if path != arg {
base.Errorf("go get %s: can't request explicit version of path in main module", arg)
continue
} now it works by |
Please fill out the complete issue template:
|
bcmills
added
modules
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
labels
Sep 4, 2019
|
consider it's resolved |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
FrozenDueToAge
modules
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
try resolves and adds dependencies by
go get -v
What did you expect to see?
success
What did you see instead?
The text was updated successfully, but these errors were encountered: