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

Unified Diff: fetch.py

Issue 13588004: fix dryrun, path check in 'fetch' (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Created 7 years, 9 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: fetch.py
diff --git a/fetch.py b/fetch.py
index 8e7b852ab0430d6b08a6e168e6b0c7f3cd274203..8316ed48e7a85e938e800f2b5ac5a7f1c3011ed3 100755
--- a/fetch.py
+++ b/fetch.py
@@ -160,6 +160,7 @@ def handle_args(argv):
if argv[1] in ('-h', '--help', 'help'):
usage()
+ dryrun = False
if argv[1] in ('-n', '--dry-run'):
dryrun = True
argv.pop(1)
@@ -180,7 +181,7 @@ def run_recipe_fetch(recipe, props, aliased=False):
"""Invoke a recipe's fetch method with the passed-through args
and return its json output as a python object."""
recipe_path = os.path.abspath(os.path.join(SCRIPT_PATH, 'recipes', recipe))
- if not os.path.exists(recipe_path):
+ if not os.path.exists(recipe_path + '.py'):
print "Could not find a recipe for %s" % recipe
sys.exit(1)
« 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