Merge pull request #236 from Lowercases/master

Move the signal setup closer to cmd.Start()
99designs · Mar 28, 2018 · 3c73304 · 3c73304
2 parents cbfcc50 + af629ad
commit 3c73304
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/cli/exec.go b/cli/exec.go
@@ -73,7 +73,6 @@ func ConfigureExecCommand(app *kingpin.Application) {
 		input.Keyring = keyringImpl
 		input.MfaPrompt = prompt.Method(GlobalFlags.PromptDriver)
 		input.Signals = make(chan os.Signal)
-		signal.Notify(input.Signals, os.Interrupt, os.Kill)
 		ExecCommand(app, input)
 		return nil
 	})
@@ -149,6 +148,7 @@ func ExecCommand(app *kingpin.Application, input ExecCommandInput) {
 	cmd.Stdin = os.Stdin
 	cmd.Stdout = os.Stdout
 	cmd.Stderr = os.Stderr
+	signal.Notify(input.Signals, os.Interrupt, os.Kill)
 
 	if err := cmd.Start(); err != nil {
 		app.Fatalf("%v", err)