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

Unified Diff: make.py

Issue 975803002: Run gyp on Win with --no-parallel -G config=$(BUILDTYPE), fix pylint (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Back to CRLF 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: make.py
diff --git a/make.py b/make.py
index 414cc9f5bb4fbf817fcfbc8c113258e9e6b4ddb0..fcf81b59463b0c1d479156eaff14cae1203484a9 100644
--- a/make.py
+++ b/make.py
@@ -61,11 +61,8 @@ def CheckWindowsEnvironment():
it displays an error message and exits.
"""
# If we already have the proper environment variables, nothing to do here.
- try:
- env_DevEnvDir = os.environ['DevEnvDir']
- return # found it, so we are done
- except KeyError:
- pass # go on and run the rest of this function
+ if os.environ.get('DevEnvDir'):
+ return
print ('\nCould not find Visual Studio environment variables.'
'\nPerhaps you have not yet run vcvars32.bat as described at'
@@ -98,7 +95,7 @@ def MakeWindows(targets):
# Run gyp_skia to prepare Visual Studio projects.
cd(SCRIPT_DIR)
- runcommand('python gyp_skia')
+ runcommand('python gyp_skia --no-parallel -G config=%s' % BUILDTYPE)
# We already built the gypfiles...
while TARGET_GYP in targets:
@@ -125,9 +122,11 @@ def Make(args):
targets = []
for arg in args:
- # If user requests "make all", chain to our explicitly-declared "everything"
- # target. See https://code.google.com/p/skia/issues/detail?id=932 ("gyp
- # automatically creates "all" target on some build flavors but not others")
+ # If user requests "make all", chain to our explicitly-declared
+ # "everything" target. See
+ # https://code.google.com/p/skia/issues/detail?id=932 ("gyp
+ # automatically creates "all" target on some build flavors but not
+ # others")
if arg == TARGET_ALL:
targets.append('everything')
elif arg == TARGET_CLEAN:
@@ -149,16 +148,16 @@ def Make(args):
sys.exit(0)
elif os.name == 'posix':
if sys.platform == 'darwin':
- print 'Mac developers should not run this script; see ' \
- 'https://skia.org/user/quick/macos'
+ print ('Mac developers should not run this script; see '
+ 'https://skia.org/user/quick/macos')
sys.exit(1)
elif sys.platform == 'cygwin':
- print 'Windows development on Cygwin is not currently supported; see ' \
- 'https://skia.org/user/quick/windows'
+ print ('Windows development on Cygwin is not currently supported; '
+ 'see https://skia.org/user/quick/windows')
sys.exit(1)
else:
- print 'Unix developers should not run this script; see ' \
- 'https://skia.org/user/quick/linux'
+ print ('Unix developers should not run this script; see '
+ 'https://skia.org/user/quick/linux')
sys.exit(1)
else:
print 'unknown platform (os.name=%s, sys.platform=%s); see %s' % (
« 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