From 5cfeaed4dec52c8edb3f33bb9cf8ec98d846c50b Mon Sep 17 00:00:00 2001 From: Zeke Lu Date: Sat, 27 Aug 2022 02:42:52 +0000 Subject: [PATCH] cmd/go/internal/modfetch: report error on failing to derive pseudo version from recent tag 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 Auto-Submit: Bryan Mills Reviewed-by: David Chase Reviewed-by: Heschi Kreinick Reviewed-by: Damien Neil TryBot-Result: Gopher Robot Run-TryBot: Bryan Mills --- src/cmd/go/internal/modfetch/coderepo.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/cmd/go/internal/modfetch/coderepo.go b/src/cmd/go/internal/modfetch/coderepo.go index 8fb0035f8c..138c00da19 100644 --- 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) }