fixes a bug where pyroscope thinks a spy is not supported when it is

petethepig · petethepig · commit 0a9da1c52180 · 2021-01-19T11:37:35.000-08:00
diff --git a/pkg/exec/cli.go b/pkg/exec/cli.go
@@ -112,11 +112,11 @@ func performChecks(spyName string) error {
 		}
 	}
 
-	if stringsContains(spy.SupportedSpies, spyName) {
+	if !stringsContains(spy.SupportedSpies, spyName) {
 		supportedSpies := supportedSpiesWithoutGospy()
 		return fmt.Errorf(
 			"Spy \"%s\" is not supported. Available spies are: %s\n%s",
-			color.BlueString("spyName"),
+			color.BlueString(spyName),
 			strings.Join(supportedSpies, ","),
 			armMessage(),
 		)

Original file line number	Diff line number	Diff line change
`@@ -112,11 +112,11 @@ func performChecks(spyName string) error {`
`112`	`112`	`}`
`113`	`113`	`}`
`114`	`114`
`115`		`- if stringsContains(spy.SupportedSpies, spyName) {`
	`115`	`+ if !stringsContains(spy.SupportedSpies, spyName) {`
`116`	`116`	`supportedSpies := supportedSpiesWithoutGospy()`
`117`	`117`	`return fmt.Errorf(`
`118`	`118`	`"Spy \"%s\" is not supported. Available spies are: %s\n%s",`
`119`		`- color.BlueString("spyName"),`
	`119`	`+ color.BlueString(spyName),`
`120`	`120`	`strings.Join(supportedSpies, ","),`
`121`	`121`	`armMessage(),`
`122`	`122`	`)`