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

Unified Diff: trychange.py

Issue 3324007: Rename retcode to returncode to be consistent with subprocess. (Closed)
Patch Set: Created 10 years, 3 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 | « tests/gclient_utils_test.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trychange.py
diff --git a/trychange.py b/trychange.py
index 7cd90b9b0696302bd6584c8bb099a763e1575645..f1ea12ddcc8a47bc45bbdec15746f3f21b1f127c 100755
--- a/trychange.py
+++ b/trychange.py
@@ -430,10 +430,10 @@ def GuessVCS(options, path):
real_path)
return GIT(options, path)
except gclient_utils.CheckCallError, e:
- if e.retcode != errno.ENOENT and e.retcode != 128:
+ if e.returncode != errno.ENOENT and e.returncode != 128:
# ENOENT == 2 = they don't have git installed.
# 128 = git error code when not in a repo.
- logging.warn(e.retcode)
+ logging.warn(e.returncode)
bradn 2010/09/03 20:31:02 Maybe a real message?
raise
raise NoTryServerAccess("Could not guess version control system. "
"Are you in a working copy directory?")
« no previous file with comments | « tests/gclient_utils_test.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698