Merge back CLI tools from v2.1.0.

Confirms that it makes v1 work with go1.16. With this:

  go get github.com/maruel/panicparse/cmd/pp

does the right thing! It leverages an updated version of cmd/ and
internal/, and then uses stack@v2 instead of stack/ here.

Found a way to hack my way through:
- Copied .github/workflows/test.yaml from v2.1.0.
- Deleted all of vendor/. It was breaking the CI tests.
- Updated stack/context_test.go to make it work with go1.16.
- Updated stack/context_test.go to make it work with all of vendor/
  removed.
- Use v2.1.0 version of the executables in cmd/ and //main.go.
- Use v2.1.0 version of the internal/.
- Copied stack@v2.1.0 back into
  vendor/github.com/maruel/panicparse/v2/stack. Since older Go versions still
  look into it, they are able to load v2 even if go modules are disabled.

Author: maruel

cmd/panic/internal/ùtf8/ùtf8.go cmd/panic/internal/utf8/ùtf8.gocmd/panic/internal/ùtf8/ùtf8.go renamed to cmd/panic/internal/utf8/ùtf8.go

@@ -37,26 +37,19 @@ import (
 
 	"github.com/maruel/panicparse/cmd/panic/internal"
 	correct "github.com/maruel/panicparse/cmd/panic/internal/incorrect"
-	"github.com/maruel/panicparse/cmd/panic/internal/ùtf8"
+	ùtf8 "github.com/maruel/panicparse/cmd/panic/internal/utf8"
 )
 
 func main() {
 	if len(os.Args) == 2 {
-		n := os.Args[1]
-		if f, ok := types[n]; ok {
-			fmt.Printf("GOTRACEBACK=%s\n", os.Getenv("GOTRACEBACK"))
-			if n == "simple" {
-				// Since the map lookup creates another call stack entry, add a one-off
-				// "simple" panic style to test the very minimal case.
-				// types["simple"].f is never called.
-				panic("simple")
-			}
-			f.f()
-			os.Exit(3)
-		}
-		// Undocumented command to do a raw dump of the supported commands. This is
-		// used by unit tests in ../../stack.
-		if n == "dump_commands" {
+		switch n := os.Args[1]; n {
+		case "-h", "-help", "--help", "help":
+			usage()
+			os.Exit(0)
+
+		case "dump_commands":
+			// Undocumented command to do a raw dump of the supported commands. This
+			// is used by unit tests in ../../stack.
 			items := make([]string, 0, len(types))
 			for n := range types {
 				items = append(items, n)
@@ -66,11 +59,25 @@ func main() {
 				fmt.Printf("%s\n", n)
 			}
 			os.Exit(0)
+
+		default:
+			if f, ok := types[n]; ok {
+				fmt.Printf("GOTRACEBACK=%s\n", os.Getenv("GOTRACEBACK"))
+				if n == "simple" {
+					// Since the map lookup creates another call stack entry, add a
+					// one-off "simple" panic style to test the very minimal case.
+					// types["simple"].f is never called.
+					panic("simple")
+				}
+				f.f()
+				os.Exit(3)
+			}
+			fmt.Fprintf(stdErr, "unknown panic style %q\n", n)
+			os.Exit(1)
 		}
-		fmt.Fprintf(stdErr, "unknown panic style %q\n", n)
-		os.Exit(1)
 	}
 	usage()
+	os.Exit(1)
 }
 
 // Mocked in test.
@@ -111,7 +118,7 @@ func panicRaceDisabled(name string) {
 
 func rerunWithFastCrash() {
 	if os.Getenv("GORACE") != "log_path=stderr halt_on_error=1" {
-		os.Setenv("GORACE", "log_path=stderr halt_on_error=1")
+		_ = os.Setenv("GORACE", "log_path=stderr halt_on_error=1")
 		c := exec.Command(os.Args[0], os.Args[1:]...)
 		c.Stderr = os.Stderr
 		if err, ok := c.Run().(*exec.ExitError); ok {
@@ -413,7 +420,7 @@ Set GOTRACEBACK before running this tool to see how it affects the panic output.
 
 Select the way to panic:
 `
-	io.WriteString(stdErr, t)
+	_, _ = io.WriteString(stdErr, t)
 	names := make([]string, 0, len(types))
 	m := 0
 	for n := range types {
@@ -426,7 +433,6 @@ Select the way to panic:
 	for _, n := range names {
 		fmt.Fprintf(stdErr, "- %-*s  %s\n", m, n, types[n].desc)
 	}
-	os.Exit(2)
 }
 
 //

cmd/panic/main.go

@@ -27,7 +27,7 @@ func GetAsync(url string) {
 		if err != nil {
 			log.Fatalf("failed to read: %v", err)
 		}
-		resp.Body.Close()
+		_ = resp.Body.Close()
 		log.Fatal("the goal is to not complete this request")
 	}()
 }
@@ -39,7 +39,7 @@ func URL1Handler(w http.ResponseWriter, req *http.Request) {
 	w.Header().Set("Content-Length", "100000")
 	w.WriteHeader(200)
 	b := [4096]byte{}
-	w.Write(b[:])
+	_, _ = w.Write(b[:])
 	<-Unblock
 }
 
@@ -50,6 +50,6 @@ func URL2Handler(w http.ResponseWriter, req *http.Request) {
 	w.Header().Set("Content-Length", "100000")
 	w.WriteHeader(200)
 	b := [4096]byte{}
-	w.Write(b[:])
+	_, _ = w.Write(b[:])
 	<-Unblock
 }

cmd/panicweb/internal/internal.go

@@ -20,16 +20,26 @@ import (
 	"net"
 	"net/http"
 	_ "net/http/pprof"
+	"os"
 	"runtime"
 	"strings"
 	"sync"
 	"time"
 
 	"github.com/maruel/panicparse/cmd/panicweb/internal"
-	"github.com/maruel/panicparse/stack/webstack"
+	"github.com/maruel/panicparse/v2/stack/webstack"
 	"github.com/mattn/go-colorable"
 )
 
+var rootPage = []byte(`<!DOCTYPE html>
+<ul>
+	<li><a href="/panicparse">/panicparse</a></li>
+	<li><a href="/debug/pprof/goroutine?debug=2">/debug/pprof/goroutine?debug=2</a></li>
+	<li><a href="/url1">/url1</a></li>
+	<li><a href="/url2">/url2</a></li>
+</ul>
+`)
+
 func main() {
 	allowremote := flag.Bool("allowremote", false, "allows access from non-localhost; implies -wait")
 	sleep := flag.Bool("wait", false, "sleep instead of crashing")
@@ -91,10 +101,22 @@ func main() {
 	} else {
 		http.HandleFunc("/panicparse", webstack.SnapshotHandler)
 	}
+	http.HandleFunc("/", func(w http.ResponseWriter, req *http.Request) {
+		w.Header().Set("Content-Type", "text/html; charset=utf-8")
+		_, _ = w.Write(rootPage)
+	})
 	go http.Serve(ln, http.DefaultServeMux)
 
 	// Start many clients.
-	url := "http://" + ln.Addr().String() + "/"
+	a := ln.Addr()
+	url := fmt.Sprintf("http://%s/", a)
+	if *allowremote {
+		if h, err := os.Hostname(); err == nil {
+			if t, ok := a.(*net.TCPAddr); ok {
+				url = fmt.Sprintf("http://%s:%d/", h, t.Port)
+			}
+		}
+	}
 	for i := 0; i < 10; i++ {
 		internal.GetAsync(url + "url1")
 	}

cmd/panicweb/main.go

@@ -9,5 +9,5 @@ package main
 import "golang.org/x/sys/unix"
 
 func sysHang() {
-	unix.Nanosleep(&unix.Timespec{Sec: 366 * 24 * 60 * 60}, &unix.Timespec{})
+	_ = unix.Nanosleep(&unix.Timespec{Sec: 366 * 24 * 60 * 60}, &unix.Timespec{})
 }

cmd/panicweb/main_unix.go

@@ -2,18 +2,10 @@
 // Use of this source code is governed under the Apache License, Version 2.0
 // that can be found in the LICENSE file.
 
-// panicparse: analyzes stack dump of Go processes and simplifies it.
+// pp: analyzes stack dump of Go processes and simplifies it.
 //
 // It is mostly useful on servers will large number of identical goroutines,
 // making the crash dump harder to read than strictly necessary.
-//
-// Colors:
-//  - Magenta: first goroutine to be listed.
-//  - Yellow: main package.
-//  - Green: standard library.
-//  - Red: other packages.
-//
-// Bright colors are used for exported symbols.
 package main
 
 import (
@@ -24,7 +16,7 @@ import (
 )
 
 func main() {
-	if err := internal.Main("/cmd/pp"); err != nil {
+	if err := internal.Main(); err != nil {
 		fmt.Fprintf(os.Stderr, "Failed: %s\n", err)
 		os.Exit(1)
 	}

cmd/pp/main.go

@@ -3,9 +3,10 @@ module github.com/maruel/panicparse
 go 1.11
 
 require (
-	github.com/google/go-cmp v0.4.0
-	github.com/mattn/go-colorable v0.1.6
+	github.com/google/go-cmp v0.5.1
+	github.com/maruel/panicparse/v2 v2.1.0
+	github.com/mattn/go-colorable v0.1.7
 	github.com/mattn/go-isatty v0.0.12
-	github.com/mgutz/ansi v0.0.0-20170206155736-9520e82c474b
-	golang.org/x/sys v0.0.0-20200223170610-d5e6a3e2c0ae
+	github.com/mgutz/ansi v0.0.0-20200706080929-d51e80ef957d
+	golang.org/x/sys v0.0.0-20200724161237-0e2f3a69832c
 )

go.mod

@@ -1,13 +1,17 @@
-github.com/google/go-cmp v0.4.0 h1:xsAVV57WRhGj6kEIi8ReJzQlHHqcBYCElAvkovg3B/4=
-github.com/google/go-cmp v0.4.0/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
-github.com/mattn/go-colorable v0.1.6 h1:6Su7aK7lXmJ/U79bYtBjLNaha4Fs1Rg9plHpcH+vvnE=
-github.com/mattn/go-colorable v0.1.6/go.mod h1:u6P/XSegPjTcexA+o6vUJrdnUu04hMope9wVRipJSqc=
+github.com/google/go-cmp v0.5.1 h1:JFrFEBb2xKufg6XkJsJr+WbKb4FQlURi5RUcBveYu9k=
+github.com/google/go-cmp v0.5.1/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
+github.com/maruel/panicparse/v2 v2.1.0 h1:Ml+NucJWyfNVA7YVXavI3EAwqGcPEY1epYjs4CSMwy0=
+github.com/maruel/panicparse/v2 v2.1.0/go.mod h1:AeTWdCE4lcq8OKsLb6cHSj1RWHVSnV9HBCk7sKLF4Jg=
+github.com/mattn/go-colorable v0.1.7 h1:bQGKb3vps/j0E9GfJQ03JyhRuxsvdAanXlT9BTw3mdw=
+github.com/mattn/go-colorable v0.1.7/go.mod h1:u6P/XSegPjTcexA+o6vUJrdnUu04hMope9wVRipJSqc=
 github.com/mattn/go-isatty v0.0.12 h1:wuysRhFDzyxgEmMf5xjvJ2M9dZoWAXNNr5LSBS7uHXY=
 github.com/mattn/go-isatty v0.0.12/go.mod h1:cbi8OIDigv2wuxKPP5vlRcQ1OAZbq2CE4Kysco4FUpU=
-github.com/mgutz/ansi v0.0.0-20170206155736-9520e82c474b h1:j7+1HpAFS1zy5+Q4qx1fWh90gTKwiN4QCGoY9TWyyO4=
-github.com/mgutz/ansi v0.0.0-20170206155736-9520e82c474b/go.mod h1:01TrycV0kFyexm33Z7vhZRXopbI8J3TDReVlkTgMUxE=
+github.com/mgutz/ansi v0.0.0-20200706080929-d51e80ef957d h1:5PJl274Y63IEHC+7izoQE9x6ikvDFZS2mDVS3drnohI=
+github.com/mgutz/ansi v0.0.0-20200706080929-d51e80ef957d/go.mod h1:01TrycV0kFyexm33Z7vhZRXopbI8J3TDReVlkTgMUxE=
 golang.org/x/sys v0.0.0-20200116001909-b77594299b42/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20200223170610-d5e6a3e2c0ae h1:/WDfKMnPU+m5M4xB+6x4kaepxRw6jWvR5iDRdvjHgy8=
 golang.org/x/sys v0.0.0-20200223170610-d5e6a3e2c0ae/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/sys v0.0.0-20200724161237-0e2f3a69832c h1:UIcGWL6/wpCfyGuJnRFJRurA+yj8RrW7Q6x2YMCXt6c=
+golang.org/x/sys v0.0.0-20200724161237-0e2f3a69832c/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543 h1:E7g+9GITq07hpfrRu66IVDexMakfv52eLZ2CXBWiKr4=
 golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=

go.sum

@@ -0,0 +1,9 @@
+// Copyright 2020 Marc-Antoine Ruel. All rights reserved.
+// Use of this source code is governed under the Apache License, Version 2.0
+// that can be found in the LICENSE file.
+
+// +build go1.13
+
+package internaltest
+
+const wrap = "%w"

internal/internaltest/go1.13.go

@@ -0,0 +1,10 @@
+// Copyright 2020 Marc-Antoine Ruel. All rights reserved.
+// Use of this source code is governed under the Apache License, Version 2.0
+// that can be found in the LICENSE file.
+
+// +build go1.1
+// +build !go1.13
+
+package internaltest
+
+const wrap = "%v"