Fix incorrect assumption that APPLE implies Mac.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Apr 2016 08:09:56 +0000 (08:09 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 18 Apr 2016 08:09:56 +0000 (08:09 +0000)
https://bugs.webkit.org/show_bug.cgi?id=156679

Addresses build failure introduced in r198947

Patch by Jeremy Huddleston Sequoia <jeremyhu@apple.com> on 2016-04-18
Reviewed by Alex Christensen.

* CMakeLists.txt:

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

Source/WebKit2/CMakeLists.txt
Source/WebKit2/ChangeLog
Tools/CMakeLists.txt
Tools/ChangeLog

index 1b172bd..97fe019 100644 (file)
@@ -868,7 +868,7 @@ if (WebKit2_NetworkProcess_OUTPUT_NAME)
     set_target_properties(NetworkProcess PROPERTIES OUTPUT_NAME ${WebKit2_NetworkProcess_OUTPUT_NAME})
 endif ()
 
-if (ENABLE_PLUGIN_PROCESS AND NOT APPLE)
+if (ENABLE_PLUGIN_PROCESS AND NOT "${PORT}" STREQUAL "Mac")
     add_definitions(-DENABLE_PLUGIN_PROCESS=1)
     add_executable(PluginProcess ${PluginProcess_SOURCES})
     add_webkit2_prefix_header(PluginProcess)
@@ -880,7 +880,7 @@ if (ENABLE_PLUGIN_PROCESS AND NOT APPLE)
     endif ()
 endif ()
 
-if (ENABLE_DATABASE_PROCESS AND NOT APPLE)
+if (ENABLE_DATABASE_PROCESS AND NOT "${PORT}" STREQUAL "Mac")
     add_definitions(-DENABLE_DATABASE_PROCESS=1)
     add_executable(DatabaseProcess ${DatabaseProcess_SOURCES})
     add_webkit2_prefix_header(DatabaseProcess)
@@ -892,7 +892,7 @@ if (ENABLE_DATABASE_PROCESS AND NOT APPLE)
     endif ()
 endif ()
 
-if (APPLE)
+if ("${PORT}" STREQUAL "Mac")
     set(_web_xpc_dir ${CMAKE_LIBRARY_OUTPUT_DIRECTORY}/WebKit.framework/Versions/A/XPCServices/com.apple.WebKit.WebContent.Development.xpc/Contents)
     make_directory(${_web_xpc_dir}/MacOS)
     make_directory(${_web_xpc_dir}/Resources)
index e5f4879..b873d8c 100644 (file)
@@ -1,3 +1,14 @@
+2016-04-18  Jeremy Huddleston Sequoia  <jeremyhu@apple.com>
+
+        Fix incorrect assumption that APPLE implies Mac.
+        https://bugs.webkit.org/show_bug.cgi?id=156679
+
+        Addresses build failure introduced in r198947
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+
 2016-04-17  Yoav Weiss  <yoav@yoav.ws>
 
         Initial Link preload support
index fae8b61..a9e79df 100644 (file)
@@ -27,6 +27,10 @@ elseif ("${PORT}" STREQUAL "GTK")
     if (ENABLE_MINIBROWSER)
         add_subdirectory(MiniBrowser/gtk)
     endif ()
+elseif ("${PORT}" STREQUAL "Mac")
+    add_subdirectory(DumpRenderTree)
+    add_subdirectory(WebKitTestRunner)
+    add_subdirectory(MiniBrowser/mac)
 endif ()
 
 if (WIN32)
@@ -35,12 +39,6 @@ if (WIN32)
     add_subdirectory(MiniBrowser/win)
 endif ()
 
-if (APPLE)
-    add_subdirectory(DumpRenderTree)
-    add_subdirectory(WebKitTestRunner)
-    add_subdirectory(MiniBrowser/mac)
-endif ()
-
 if (ENABLE_WEBKIT2 AND ENABLE_API_TESTS)
     add_subdirectory(TestWebKitAPI)
 endif ()
index 3586b3c..498d1ee 100644 (file)
@@ -1,3 +1,14 @@
+2016-04-18  Jeremy Huddleston Sequoia  <jeremyhu@apple.com>
+
+        Fix incorrect assumption that APPLE implies Mac.
+        https://bugs.webkit.org/show_bug.cgi?id=156679
+
+        Addresses build failure introduced in r198947
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+
 2016-04-18  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK] Test /webkit2/WebKitWebView/mouse-target fails with overlay scrollbars