gperf is only used in WebCore
authoryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 30 Dec 2018 21:23:11 +0000 (21:23 +0000)
committeryusukesuzuki@slowstart.org <yusukesuzuki@slowstart.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 30 Dec 2018 21:23:11 +0000 (21:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193061

Reviewed by Don Olmstead.

gperf is only used in WebCore. So if the port does not have WebCore,
it is not necessary. For example, JSCOnly port does not require it.
This patch removes gperf dependency in these ports by checking
ENABLE_WEBCORE.

* Source/cmake/WebKitCommon.cmake:

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

ChangeLog
Source/cmake/WebKitCommon.cmake

index bf4a5b1..4d3b897 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2018-12-30  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
+
+        gperf is only used in WebCore
+        https://bugs.webkit.org/show_bug.cgi?id=193061
+
+        Reviewed by Don Olmstead.
+
+        gperf is only used in WebCore. So if the port does not have WebCore,
+        it is not necessary. For example, JSCOnly port does not require it.
+        This patch removes gperf dependency in these ports by checking
+        ENABLE_WEBCORE.
+
+        * Source/cmake/WebKitCommon.cmake:
+
 2018-12-28  Yusuke Suzuki  <yusukesuzuki@slowstart.org>
 
         Add ENABLE_UNIFIED_BUILDS option to cmake ports
index cafb66a..97b9f20 100644 (file)
@@ -13,9 +13,6 @@ if (NOT HAS_RUN_WEBKIT_COMMON)
         list(APPEND CMAKE_PROGRAM_PATH $ENV{SystemDrive}/cygwin/bin)
     endif ()
 
-    # TODO Enforce version requirement for gperf
-    find_package(Gperf 3.0.1 REQUIRED)
-
     # TODO Enforce version requirement for perl
     find_package(Perl 5.10.0 REQUIRED)
     find_package(PerlModules COMPONENTS JSON::PP REQUIRED)
@@ -58,6 +55,13 @@ if (NOT HAS_RUN_WEBKIT_COMMON)
     include(OptionsCommon)
     include(Options${PORT})
 
+    # Check gperf after including OptionsXXX.cmake since gperf is required only when ENABLE_WEBCORE is true,
+    # and ENABLE_WEBCORE is configured in OptionsXXX.cmake.
+    if (ENABLE_WEBCORE)
+        # TODO Enforce version requirement for gperf
+        find_package(Gperf 3.0.1 REQUIRED)
+    endif ()
+
     # -----------------------------------------------------------------------------
     # Job pool to avoid running too many memory hungry linker processes
     # -----------------------------------------------------------------------------