From fda5dcd9bba38bcdbe9ac473985e108e125c34a5 Mon Sep 17 00:00:00 2001 From: Michael Matloob Date: Thu, 6 Oct 2022 13:41:02 -0400 Subject: [PATCH] cmd/go: fix incorrect determination of import path is DirImportPath In practice this only shows up when a vendored package, imported on the command line, imports an internal package. Change-Id: I34c161d1f1ef15a87c58a422f17d11f77fbac53f Reviewed-on: https://go-review.googlesource.com/c/go/+/439735 Run-TryBot: Michael Matloob Reviewed-by: Michael Matloob TryBot-Result: Gopher Robot Reviewed-by: Bryan Mills --- src/cmd/go/internal/modload/load.go | 1 + src/cmd/go/testdata/script/vendor_internal.txt | 16 ++++++++++++++++ 2 files changed, 17 insertions(+) create mode 100644 src/cmd/go/testdata/script/vendor_internal.txt diff --git a/src/cmd/go/internal/modload/load.go b/src/cmd/go/internal/modload/load.go index e74780a7b2..afd6c80370 100644 --- a/src/cmd/go/internal/modload/load.go +++ b/src/cmd/go/internal/modload/load.go @@ -753,6 +753,7 @@ func (mms *MainModuleSet) DirImportPath(ctx context.Context, dir string) (path s suffix := filepath.ToSlash(dir[len(modRoot):]) if strings.HasPrefix(suffix, "/vendor/") { longestPrefixPath = strings.TrimPrefix(suffix, "/vendor/") + continue } longestPrefixPath = mms.PathPrefix(v) + suffix } diff --git a/src/cmd/go/testdata/script/vendor_internal.txt b/src/cmd/go/testdata/script/vendor_internal.txt new file mode 100644 index 0000000000..4c0f1facee --- /dev/null +++ b/src/cmd/go/testdata/script/vendor_internal.txt @@ -0,0 +1,16 @@ +go build ./vendor/foo.com/internal/bar/a + +-- go.mod -- +module example.com/x +go 1.19 + +require "foo.com/internal/bar" v1.0.0 +-- vendor/modules.txt -- +# foo.com/internal/bar v1.0.0 +## explicit +foo.com/internal/bar/a +-- vendor/foo.com/internal/bar/a/a.go -- +package a +import _ "foo.com/internal/bar/b" +-- vendor/foo.com/internal/bar/b/b.go -- +package b \ No newline at end of file -- 2.48.1