cmd/dist: mark linux/sparc64 as a broken port, remove incomplete map

The linux/sparc64 port is incomplete—it doesn't work, and it doesn't
have a builder. Now that dist supports broken ports, mark it as such.

The incomplete map was created to hide ports that aren't functional
from dist list output. Now that we have the broken port concept, it
seems largely redundant, so remove it for now.

For #56679.
Updates #28944.

Change-Id: I34bd23e913ed6d786a4d0aa8d2852f2b926fe4b6
Reviewed-on: https://go-review.googlesource.com/c/go/+/458516
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>

diff --git a/src/cmd/dist/build.go b/src/cmd/dist/build.go
index 96cbd3dbf6c9795ed1381ad32b187b64cad3b523..75daf7fdbeb4f15f2593c8b75ece1ae281cdb27f 100644 (file)
--- a/src/cmd/dist/build.go
+++ b/src/cmd/dist/build.go
@@ -1681,17 +1681,13 @@ var cgoEnabled = map[string]bool{
        "windows/arm64":   true,
 }
 
-// List of platforms which are supported but not complete yet. These get
-// filtered out of cgoEnabled for 'dist list'. See go.dev/issue/28944.
-var incomplete = map[string]bool{
-       "linux/sparc64": true,
-}
-
 // List of platforms that are marked as broken ports.
 // These require -force flag to build, and also
 // get filtered out of cgoEnabled for 'dist list'.
 // See go.dev/issue/56679.
-var broken = map[string]bool{}
+var broken = map[string]bool{
+       "linux/sparc64": true, // An incomplete port. See CL 132155.
+}
 
 // List of platforms which are first class ports. See go.dev/issue/38874.
 var firstClass = map[string]bool{
@@ -1839,7 +1835,7 @@ func cmdlist() {
 
        var plats []string
        for p := range cgoEnabled {
-               if broken[p] || incomplete[p] {
+               if broken[p] {
                        continue
                }
                plats = append(plats, p)

diff --git a/src/cmd/dist/test.go b/src/cmd/dist/test.go
index 5b725394e48d3b267447386ab723a08ad8f5e10a..6eca06859571701d8eb99a5f443feb0bc62a3a3a 100644 (file)
--- a/src/cmd/dist/test.go
+++ b/src/cmd/dist/test.go
@@ -265,12 +265,6 @@ func (t *tester) run() {
        if t.failed {
                fmt.Println("\nFAILED")
                xexit(1)
-       } else if incomplete[goos+"/"+goarch] {
-               // The test succeeded, but consider it as failed so we don't
-               // forget to remove the port from the incomplete map once the
-               // port is complete.
-               fmt.Println("\nFAILED (incomplete port)")
-               xexit(1)
        } else if t.partial {
                fmt.Println("\nALL TESTS PASSED (some were excluded)")
        } else {