Small refactor

exec.go

@@ -81,43 +81,46 @@ func (h *HookExec) Exec(timeout time.Duration) error {
 	defer h.HookServer.Unlock()
 
 	for _, f := range files {
-		cmd := exec.Command(f)
+		err := execFile(f, h, timeout)
+		multierror.Append(result, err)
+	}
 
-		cmd.Stdout = os.Stdout
-		cmd.Stderr = os.Stderr
+	return result
+}
 
-		stdin, err := cmd.StdinPipe()
-		if err != nil {
-			multierror.Append(result, err)
-			continue
-		}
-		defer stdin.Close()
+func execFile(f string, h *HookExec, timeout time.Duration) error {
+	cmd := exec.Command(f)
 
-		// io.Copy( cmd.StdinPipe
-		err = cmd.Start()
-		if err != nil {
-			multierror.Append(result, err)
-			continue
-		}
+	cmd.Stdout = os.Stdout
+	cmd.Stderr = os.Stderr
+
+	stdin, err := cmd.StdinPipe()
+	if err != nil {
+		return err
+	}
+	defer stdin.Close()
 
-		h.Data.Seek(0, 0)
-		io.Copy(stdin, h.Data)
-		stdin.Close()
+	err = cmd.Start()
+	if err != nil {
+		return err
+	}
 
-		timer := time.AfterFunc(timeout, func() {
-			cmd.Process.Kill()
-		})
+	h.Data.Seek(0, 0)
+	io.Copy(stdin, h.Data)
+	stdin.Close()
 
-		err = cmd.Wait()
-		timer.Stop()
+	timer := time.AfterFunc(timeout, func() {
+		cmd.Process.Kill()
+	})
 
-		if err != nil {
-			multierror.Append(result, err)
-			continue
-		}
+	err = cmd.Wait()
+	timer.Stop()
+
+	if err != nil {
+		return err
 	}
 
-	return result
+	return nil
 }
 
 // todo: base this on OS