cmd/go/internal/modfetch: report error on failing to derive pseudo version from recen...

The current implementation ignores the error when it tries to get
the recent tag on revisions, which results in incorrect pseudo
version (v0.0.0-) is derived.

Fixes #53935

Change-Id: I153d851eb913fb7e40051e194c92b9ca5bf0e906
GitHub-Last-Rev: 6ba1d90df56cc0f30557863aa8418c54885abb14
GitHub-Pull-Request: golang/go#54701
Reviewed-on: https://go-review.googlesource.com/c/go/+/426079
Reviewed-by: Bryan Mills <bcmills@google.com>
Auto-Submit: Bryan Mills <bcmills@google.com>
Reviewed-by: David Chase <drchase@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Bryan Mills <bcmills@google.com>

diff --git a/src/cmd/go/internal/modfetch/coderepo.go b/src/cmd/go/internal/modfetch/coderepo.go
index 8fb0035f8c0f608f5afeac436ee86ddf18dd267b..138c00da19e79888f0eb358c5e8a13c0bdee73b6 100644 (file)
--- a/src/cmd/go/internal/modfetch/coderepo.go
+++ b/src/cmd/go/internal/modfetch/coderepo.go
@@ -607,7 +607,10 @@ func (r *codeRepo) convert(info *codehost.RevInfo, statVers string) (*RevInfo, e
                return !isRetracted(v)
        }
        if pseudoBase == "" {
-               tag, _ := r.code.RecentTag(info.Name, tagPrefix, tagAllowed)
+               tag, err := r.code.RecentTag(info.Name, tagPrefix, tagAllowed)
+               if err != nil {
+                       return nil, err
+               }
                if tag != "" {
                        pseudoBase, _ = tagToVersion(tag)
                }