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

Unified Diff: trychange.py

Issue 172099: Unbreak `gcl try` if EMAIL_ADDRESS is not set. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools/
Patch Set: Created 11 years, 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trychange.py
===================================================================
--- trychange.py (revision 25111)
+++ trychange.py (working copy)
@@ -312,9 +312,11 @@
try:
# Don't use '--non-interactive', since we want it to prompt for
# crendentials if necessary.
- RunCommand(['svn', 'checkout', '--depth', 'empty',
- '--username', options.email,
- options.svn_repo, temp_dir])
+ command = ['svn', 'checkout', '--depth', 'empty',
+ options.svn_repo, temp_dir]
Mark Mentovai 2009/09/02 04:01:44 Pull this to the left one space now to keep it lin
Nico 2009/09/02 04:05:03 Done.
+ if options.email:
+ command += ['--username', options.email]
+ RunCommand(command)
# TODO(maruel): Use a subdirectory per user?
current_time = str(datetime.datetime.now()).replace(':', '.')
file_name = (EscapeDot(options.user) + '.' + EscapeDot(options.name) +
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698