Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: git_number.py

Issue 955993006: Handle KeyboardInterrupt consistently in python scripts (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « git_new_branch.py ('k') | git_rebase_update.py » ('j') | git_upstream_diff.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: git_number.py
diff --git a/git_number.py b/git_number.py
index 6997f51b3d0ffd6a21383949315ff7dbdcd6109d..f4776e10c7e29a8134c1dfec8774b5b4733e14c4 100755
--- a/git_number.py
+++ b/git_number.py
@@ -277,7 +277,12 @@ def main(): # pragma: no cover
return 0
except KeyboardInterrupt:
M-A Ruel 2015/02/26 00:57:39 It's already handled there, worth removing
Sam Clegg 2015/02/26 17:29:21 Done.
return 1
+ return 0
if __name__ == '__main__': # pragma: no cover
- sys.exit(main())
+ try:
+ sys.exit(main())
+ except KeyboardInterrupt:
+ print 'interrupted'
+ sys.exit(1)
« no previous file with comments | « git_new_branch.py ('k') | git_rebase_update.py » ('j') | git_upstream_diff.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698