Fix a data race between Cmd.Run and Process.Kill (#31)

* Fix a data race between Cmd.Run and Process.Kill

* Travis: test with race detection enabled

.travis.yml

@@ -16,7 +16,7 @@ env:
 install:
   - go get -u github.com/kardianos/govendor
   - go install github.com/kardianos/govendor
-  - go build -o $GOPATH/bin/goodman github.com/snikch/goodman/cmd/goodman
+  - go build -race -o $GOPATH/bin/goodman github.com/snikch/goodman/cmd/goodman
   - cd example && $GOPATH/bin/govendor sync && cd ..
 
 go_import_path: github.com/snikch/goodman

cmd/goodman/main.go

@@ -34,7 +34,7 @@ func main() {
 		closeHooksServers()
 		os.Exit(0)
 	}()
-	hooksServerCount = len(args) - 1
+	hooksServerCount = len(hookPaths)
 	if len(args) < 2 {
 		runners = append(runners, &goodman.DummyRunner{})
 	} else {
@@ -45,11 +45,13 @@ func main() {
 			fmt.Println("Sending to channel\n")
 			cmds <- cmd
 			fmt.Println("Completed")
+			log.Printf("Starting hooks server")
+			if err := cmd.Start(); err != nil {
+				panic("Failed to start hooks server " + err.Error())
+			}
 			go func() {
-				log.Printf("Starting hooks server in go routine")
-				err := cmd.Run()
-				if err != nil {
-					fmt.Println("Hooks client failed with " + err.Error())
+				if err := cmd.Wait(); err != nil {
+					fmt.Println("Hooks server failed with " + err.Error())
 				}
 			}()
 			rpcService := reflect.TypeOf((*hooks.HooksRunner)(nil)).Elem().Name()