Document any crashes from the user's text editor
Rather than failing with no information, display the child exit status and the command line we tried to use to edit a text file. There may be some useful information to help understand the crash. Signed-off-by: Shawn O. Pearce <sop@google.com>
This commit is contained in:
parent
fb5c8fd948
commit
54fccd71fb
2 changed files with 14 additions and 2 deletions
11
editor.py
11
editor.py
|
@ -76,8 +76,15 @@ least one of these before using this command."""
|
|||
os.close(fd)
|
||||
fd = None
|
||||
|
||||
if subprocess.Popen(editor + [path]).wait() != 0:
|
||||
raise EditorError()
|
||||
try:
|
||||
rc = subprocess.Popen(editor + [path]).wait()
|
||||
except OSError, e:
|
||||
raise EditorError('editor failed, %s: %s %s'
|
||||
% (str(e), cls._GetEditor(), path))
|
||||
if rc != 0:
|
||||
raise EditorError('editor failed with exit status %d: %s %s'
|
||||
% (rc, cls._GetEditor(), path))
|
||||
|
||||
fd2 = open(path)
|
||||
try:
|
||||
return fd2.read()
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue