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

Unified Diff: make.py

Issue 206463007: change default build (in "make" wrapper) to ninja on all platforms (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: rebase Created 6 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 | « gyp_skia ('k') | platform_tools/chromeos/bin/chromeos_setup.sh » ('j') | 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 28f3554a492076fbeeb3a532dccca54fcf9061a4..306a977c4301afa87f355734f279e13f08e01a4c 100644
--- a/make.py
+++ b/make.py
@@ -42,11 +42,6 @@ def rmtree(path):
print '> rmtree %s' % path
shutil.rmtree(path, ignore_errors=True)
-def mkdirs(path):
- print '> mkdirs %s' % path
- if not os.path.isdir(path):
- os.makedirs(path)
-
def runcommand(command):
print '> %s' % command
if os.system(command):
@@ -99,30 +94,21 @@ def MakeWindows(targets):
parameters:
targets: build targets as a list of strings
"""
+ # TODO(epoger): I'm not sure if this is needed for ninja builds.
CheckWindowsEnvironment()
# Run gyp_skia to prepare Visual Studio projects.
cd(SCRIPT_DIR)
runcommand('python gyp_skia')
- # Prepare final output dir into which we will copy all binaries.
- msbuild_working_dir = os.path.join(SCRIPT_DIR, OUT_SUBDIR, GYP_SUBDIR)
- msbuild_output_dir = os.path.join(msbuild_working_dir, BUILDTYPE)
- final_output_dir = os.path.join(SCRIPT_DIR, OUT_SUBDIR, BUILDTYPE)
- mkdirs(final_output_dir)
+ # We already built the gypfiles...
+ while TARGET_GYP in targets:
+ targets.remove(TARGET_GYP)
- for target in targets:
- # We already built the gypfiles...
- if target == TARGET_GYP:
- continue
-
- cd(msbuild_working_dir)
- runcommand(
- ('msbuild /nologo /property:Configuration=%s'
- ' /target:%s /verbosity:minimal %s.sln') % (
- BUILDTYPE, target, target))
- runcommand('xcopy /y %s\* %s' % (
- msbuild_output_dir, final_output_dir))
+ # And call ninja to do the work!
+ if targets:
+ runcommand('ninja -C %s %s' % (
+ os.path.join(OUT_SUBDIR, BUILDTYPE), ' '.join(targets)))
def Make(args):
@@ -184,5 +170,3 @@ def Make(args):
# main()
Make(sys.argv[1:])
-
-
« no previous file with comments | « gyp_skia ('k') | platform_tools/chromeos/bin/chromeos_setup.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698