BUG: fix double execution of processes (#37)

pierresouchay · web-flow · commit 671c94719268 · 2020-04-16T14:50:01.000+02:00
`cmd.CombinedOutput()` already start the process, there is not need
for Run()
diff --git a/haproxy/haproxy_cmd/run.go b/haproxy/haproxy_cmd/run.go
@@ -113,10 +113,6 @@ func execAndCapture(path string, re *regexp.Regexp) (string, error) {
 	if err != nil {
 		return "", fmt.Errorf("Failed executing %s: %s", path, err.Error())
 	}
-	err = cmd.Run()
-	if err != nil {
-		return "", fmt.Errorf("Failed executing %s: %s", path, err)
-	}
 	return string(re.Find([]byte(out))), nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -113,10 +113,6 @@ func execAndCapture(path string, re *regexp.Regexp) (string, error) {`
`113`	`113`	`if err != nil {`
`114`	`114`	`return "", fmt.Errorf("Failed executing %s: %s", path, err.Error())`
`115`	`115`	`}`
`116`		`- err = cmd.Run()`
`117`		`- if err != nil {`
`118`		`- return "", fmt.Errorf("Failed executing %s: %s", path, err)`
`119`		`- }`
`120`	`116`	`return string(re.Find([]byte(out))), nil`
`121`	`117`	`}`
`122`	`118`