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

Unified Diff: gyp_skia

Issue 1716493003: gyp_skia: use path python, be quiet (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: fix Created 4 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gyp_skia
diff --git a/gyp_skia b/gyp_skia
index e68ac16a9a2f76c520feb3baa4d49b0c64116e72..91d93ca97fd3d8765915912ddb5dd4b8d887d683 100755
--- a/gyp_skia
+++ b/gyp_skia
@@ -1,4 +1,4 @@
-#!/usr/bin/python
+#!/usr/bin/env python
# Copyright 2011 The Android Open Source Project
#
@@ -78,8 +78,6 @@ if __name__ == '__main__':
args = sys.argv[1:]
if not os.getenv(ENVVAR_GYP_GENERATORS):
- print ('%s environment variable not set, using default' %
- ENVVAR_GYP_GENERATORS)
if sys.platform.startswith('darwin'):
default_gyp_generators = 'ninja,xcode'
elif sys.platform.startswith('win'):
@@ -89,7 +87,8 @@ if __name__ == '__main__':
else:
default_gyp_generators = 'ninja'
os.environ[ENVVAR_GYP_GENERATORS] = default_gyp_generators
- print '%s is "%s"' % (ENVVAR_GYP_GENERATORS, os.getenv(ENVVAR_GYP_GENERATORS))
+ print ('%s environment variable not set, using default, %s' %
+ (ENVVAR_GYP_GENERATORS, default_gyp_generators))
vs2013_runtime_dll_dirs = None
if os.getenv('CHROME_HEADLESS', '0') == '1':
@@ -148,9 +147,6 @@ if __name__ == '__main__':
# just turn that feature off.
os.environ['GYP_LINK_CONCURRENCY'] = '9001'
- print 'Updating projects from gyp files...'
- sys.stdout.flush()
-
if '--dry-run' in args:
args.remove('--dry-run')
print gyp_source_dir, ' '.join(args)
« 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