message/pipeline: sort maps to generate predictable output

When using gotext utility, the output files are always reshuffled,
which makes any kind of source versioning and incremental translation
a real challenge.

This commit sorts the Go files by package name and order index (file
name is not available) before processing the string extraction.

Furthermore, an issue with sorting plural cases has been resolved.
Originally, it sorted ["two", "few", "one", "other"] wrong.

Fixes golang/go#33552

Change-Id: I76fc5d40cf4f989e01ba6d897c0a69029ca30337
Reviewed-on: https://go-review.googlesource.com/c/text/+/207281
Run-TryBot: Jason A. Donenfeld <[email protected]>
Run-TryBot: Marcel van Lohuizen <[email protected]>
TryBot-Result: Gobot Gobot <[email protected]>
Reviewed-by: Jason A. Donenfeld <[email protected]>
Reviewed-by: Marcel van Lohuizen <[email protected]>

message/pipeline/extract.go

@@ -14,6 +14,7 @@ import (
 	"go/token"
 	"go/types"
 	"path/filepath"
+	"sort"
 	"strings"
 	"unicode"
 	"unicode/utf8"
@@ -509,8 +510,14 @@ func (px packageExtracter) getComment(n ast.Node) string {
 
 func (x *extracter) extractMessages() {
 	prog := x.iprog
+	keys := make([]*types.Package, 0, len(x.iprog.AllPackages))
+	for k := range x.iprog.AllPackages {
+		keys = append(keys, k)
+	}
+	sort.Slice(keys, func(i, j int) bool { return keys[i].Path() < keys[j].Path() })
 	files := []packageExtracter{}
-	for _, info := range x.iprog.AllPackages {
+	for _, k := range keys {
+		info := x.iprog.AllPackages[k]
 		for _, f := range info.Files {
 			// Associate comments with nodes.
 			px := packageExtracter{

message/pipeline/generate.go

@@ -271,8 +271,11 @@ func assembleSelect(m *Message, s *Select) (msg catmsg.Message, err error) {
 func sortCases(cases []string) {
 	// TODO: implement full interface.
 	sort.Slice(cases, func(i, j int) bool {
-		if cases[j] == "other" && cases[i] != "other" {
+		switch {
+		case cases[i] != "other" && cases[j] == "other":
 			return true
+		case cases[i] == "other" && cases[j] != "other":
+			return false
 		}
 		// the following code relies on '<' < '=' < any letter.
 		return cmpNumeric(cases[i], cases[j]) == -1