golang/delve
1
0
Fork 0
mirror of https://github.com/go-delve/delve.git synced 2025-10-28 12:47:22 +08:00
Code Issues Packages Projects Releases Wiki Activity

pkg/terminal: Fix exit status

Browse Source 
During a debug session if the process exited and then the user quit the
debug session, the process exit message would display again and Delve
would exit non-zero (specifically with exit code 1) despite nothing
going wrong.

This patch fixes this so that Delve exits with a clean 0 status and the
process exit message is not printed yet again.
This commit is contained in:
Derek Parker
2019-12-02 08:16:18 -08:00
committed by Alessandro Arzilli
parent 7e2660e7de
commit c119e40c6d
2 changed files with 25 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
pkg/terminal/command_test.go
View File

@ -281,6 +281,19 @@ func TestIssue411(t *testing.T) {
})
}
func TestExitStatus(t *testing.T) {
withTestTerminal("continuetestprog", t, func(term *FakeTerminal) {
term.Exec("continue")
status, err := term.handleExit()
if err != nil {
t.Fatal(err)
}
if status != 0 {
t.Fatalf("incorrect exit status, expected 0, got %d", status)
}
})
}
func TestScopePrefix(t *testing.T) {
if runtime.GOARCH == "arm64" {
t.Skip("arm64 does not support Stacktrace for now")

5
pkg/terminal/terminal.go
View File

@ -354,7 +354,8 @@ func (t *Term) handleExit() (int, error) {
s, err := t.client.GetState()
if err != nil {
if isErrProcessExited(err) && t.client.IsMulticlient() {
if isErrProcessExited(err) {
if t.client.IsMulticlient() {
answer, err := yesno(t.line, "Remote process has exited. Would you like to kill the headless instance? [Y/n] ")
if err != nil {
return 2, io.EOF
@ -366,6 +367,8 @@ func (t *Term) handleExit() (int, error) {
}
return 0, err
}
return 0, nil
}
return 1, err
}
if !s.Exited {