[CMake] Enable building WebKit2-only build for the EFL port
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Dec 2012 15:20:51 +0000 (15:20 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 3 Dec 2012 15:20:51 +0000 (15:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=103820

Patch by Laszlo Gombos <l.gombos@samsung.com> on 2012-12-03
Reviewed by Gyuyoung Kim.

.:

To create a WebKit2-only build for EFL use the following command:
build-webkit --efl --cmakeargs="-DENABLE_WEBKIT=OFF".

* CMakeLists.txt: Turn on WebKit1 support if it is not explicitly set (enabled or disabled) for
all CMake based ports.

* Source/cmake/OptionsEfl.cmake: Enable WebKit2
for the EFL port if it is not explicitly set (enabled or disabled).

Tools:

Set ENABLE_WEBKIT2 to OFF if build-webkit was run with --no-webkit2.
Never set ENABLE_WEBKIT in build-webkit. This will simplyfy invoking
cmake directly. Initialize $cmakeArgs to avoid potential
warnings when running the script.

* Scripts/build-webkit:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@136398 268f45cc-cd09-0410-ab3c-d52691b4dbfc

CMakeLists.txt
ChangeLog
Source/cmake/OptionsEfl.cmake
Tools/ChangeLog
Tools/Scripts/build-webkit

index a91f255..8f8378e 100644 (file)
@@ -13,7 +13,7 @@ ENDIF ()
 # -----------------------------------------------------------------------------
 SET(ENABLE_WEBCORE ON)
 
-IF (NOT ENABLE_WEBKIT AND NOT ENABLE_WEBKIT2)
+IF (NOT DEFINED ENABLE_WEBKIT)
     SET(ENABLE_WEBKIT ON)
 ENDIF ()
 
index cc5d68e..27af119 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2012-12-03  Laszlo Gombos  <l.gombos@samsung.com>
+
+        [CMake] Enable building WebKit2-only build for the EFL port
+        https://bugs.webkit.org/show_bug.cgi?id=103820
+
+        Reviewed by Gyuyoung Kim.
+
+        To create a WebKit2-only build for EFL use the following command:
+        build-webkit --efl --cmakeargs="-DENABLE_WEBKIT=OFF".
+
+        * CMakeLists.txt: Turn on WebKit1 support if it is not explicitly set (enabled or disabled) for
+        all CMake based ports.
+
+        * Source/cmake/OptionsEfl.cmake: Enable WebKit2 
+        for the EFL port if it is not explicitly set (enabled or disabled).
+
 2012-12-03  Zeno Albisser  <zeno@webkit.org>
 
         [Qt][Mac] QtWebKitWidgets has wrong install_name.
index fe37fbf..31e5ae4 100644 (file)
@@ -5,6 +5,10 @@ SET(PROJECT_VERSION ${PROJECT_VERSION_MAJOR}.${PROJECT_VERSION_MINOR}.${PROJECT_
 
 ADD_DEFINITIONS(-DBUILDING_EFL__=1)
 
+IF (NOT DEFINED ENABLE_WEBKIT2)
+    SET(ENABLE_WEBKIT2 ON)
+ENDIF ()
+
 FIND_PACKAGE(Cairo 1.10.2 REQUIRED)
 FIND_PACKAGE(Fontconfig 2.8.0 REQUIRED)
 FIND_PACKAGE(Sqlite REQUIRED)
index 97ed87c..fecfebb 100644 (file)
@@ -1,3 +1,17 @@
+2012-12-03  Laszlo Gombos  <l.gombos@samsung.com>
+
+        [CMake] Enable building WebKit2-only build for the EFL port
+        https://bugs.webkit.org/show_bug.cgi?id=103820
+
+        Reviewed by Gyuyoung Kim.
+
+        Set ENABLE_WEBKIT2 to OFF if build-webkit was run with --no-webkit2.
+        Never set ENABLE_WEBKIT in build-webkit. This will simplyfy invoking
+        cmake directly. Initialize $cmakeArgs to avoid potential
+        warnings when running the script.
+
+        * Scripts/build-webkit:
+
 2012-12-03  Tommy Widenflycht  <tommyw@google.com>
 
         Speech Recognition API: Update SpeechRecognitionEvent to match the specification
index e7d5646..3175346 100755 (executable)
@@ -57,7 +57,7 @@ my $installHeaders;
 my $installLibs;
 my $prefixPath;
 my $makeArgs = "";
-my $cmakeArgs;
+my $cmakeArgs = "";
 my $onlyWebKitProject = 0;
 my $noWebKit2 = 0;
 my $coverageSupport = 0;
@@ -319,8 +319,7 @@ if (isChromium()) {
 if (isEfl()) {
     # By default we build using all of the available CPUs.
     $makeArgs .= ($makeArgs ? " " : "") . "-j" . numberOfCPUs() if $makeArgs !~ /-j\s*\d+/;
-    $cmakeArgs .= ($cmakeArgs ? " " : "") . "-DENABLE_WEBKIT=ON";
-    $cmakeArgs .= " -DENABLE_WEBKIT2=ON" if !$noWebKit2;
+    $cmakeArgs = "-DENABLE_WEBKIT2=OFF " . $cmakeArgs if $noWebKit2;
 
     # We remove CMakeCache to avoid the bots to reuse cached flags when
     # we enable new features. This forces a reconfiguration.