Fix the macOS WebKit CMake build
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Sep 2017 04:19:22 +0000 (04:19 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 26 Sep 2017 04:19:22 +0000 (04:19 +0000)
* CMakeLists.txt:
Before r222194, this -umbrella flag mentioned "WebKit" twice:
first, the name of the target to affect; second, the name of the
umbrella framework. In r222194, the targets were renamed. It turns
out that a global find and replace isn't a great way to do this,
because it caught the name of the umbrella framework too.

Put it back; the umbrella framework will always be named "WebKit".

This makes it once again possible to link WebKit (before, it was
complaining about us linking directly to WebKitLegacy instead of the
nonsensical WebKitLegacy umbrella framework).

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

Source/WebKitLegacy/CMakeLists.txt
Source/WebKitLegacy/ChangeLog

index c3ef489..bdbe9e0 100644 (file)
@@ -52,7 +52,7 @@ if (TARGET WebKitLegacyGUID)
 endif ()
 
 if (APPLE)
-    set_target_properties(WebKitLegacy PROPERTIES LINK_FLAGS "-umbrella WebKitLegacy")
+    set_target_properties(WebKitLegacy PROPERTIES LINK_FLAGS "-umbrella WebKit")
 endif ()
 
 if (MSVC)
index fad3bba..44ac719 100644 (file)
@@ -1,3 +1,20 @@
+2017-09-25  Timothy Horton  <timothy_horton@apple.com>
+
+        Fix the macOS WebKit CMake build
+
+        * CMakeLists.txt:
+        Before r222194, this -umbrella flag mentioned "WebKit" twice:
+        first, the name of the target to affect; second, the name of the
+        umbrella framework. In r222194, the targets were renamed. It turns
+        out that a global find and replace isn't a great way to do this,
+        because it caught the name of the umbrella framework too.
+
+        Put it back; the umbrella framework will always be named "WebKit".
+
+        This makes it once again possible to link WebKit (before, it was
+        complaining about us linking directly to WebKitLegacy instead of the
+        nonsensical WebKitLegacy umbrella framework).
+
 2017-09-25  Chris Dumez  <cdumez@apple.com>
 
         Make StoredCredentials an enum class and rename its header