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

Unified Diff: cmake/cmake_build

Issue 1341763003: cmake_build: support SKIA_OUT and BUILDTYPE (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix Debug build. Created 5 years, 3 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 | « cmake/CMakeLists.txt ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cmake/cmake_build
diff --git a/cmake/cmake_build b/cmake/cmake_build
index 2ff7a9c541bf5895c309358b8313f6d12612f4e1..983b1e8520d1130fe2f216c1a30a39977f4a8b39 100755
--- a/cmake/cmake_build
+++ b/cmake/cmake_build
@@ -10,9 +10,17 @@
set -e
set -x
-here=`dirname $0`
+fulldir() {
+ cd `dirname $1`
+ echo `pwd`
+}
+
+here=`fulldir $0`
cores=32
+type=${BUILDTYPE:-"Release"}
+out=${SKIA_OUT:-$here}
borenet 2015/09/14 18:16:00 I think "out" is a better default.
+
echo "Bootstrapping CMake"
pushd $here/../third_party/externals/cmake
./bootstrap --parallel=$cores
@@ -20,7 +28,8 @@ make -j $cores
popd
echo "Building with bootstrapped CMake"
-pushd $here
-../third_party/externals/cmake/bin/cmake . -G Ninja
+mkdir -p $out
+pushd $out
+$here/../third_party/externals/cmake/bin/cmake -DCMAKE_BUILD_TYPE=$type $here -G Ninja
borenet 2015/09/14 18:16:00 What's the second $here used for? Should it be $o
ninja skia # The default also builds `example`, which I'm leaving a TODO for now.
popd
« no previous file with comments | « cmake/CMakeLists.txt ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698