Skip to content

Commit df598ea

Browse files
Fix the Tools build with CMake on macOS
https://bugs.webkit.org/show_bug.cgi?id=177030 Reviewed by Geoffrey Garen. Source/WebKit: * PlatformMac.cmake: Add some WebKitLegacy headers to forward (for use by the tools). Source/WebKitLegacy: * PlatformMac.cmake: Add some forwarding headers. Tools: * TestWebKitAPI/CMakeLists.txt: Add a workaround identical to what the gtest CMakeLists has. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@222131 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent d06ae0b commit df598ea

File tree

6 files changed

+41
-2
lines changed

6 files changed

+41
-2
lines changed

Source/WebKit/ChangeLog

+10
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
2017-09-16 Tim Horton <[email protected]>
2+
3+
Fix the Tools build with CMake on macOS
4+
https://bugs.webkit.org/show_bug.cgi?id=177030
5+
6+
Reviewed by Geoffrey Garen.
7+
8+
* PlatformMac.cmake:
9+
Add some WebKitLegacy headers to forward (for use by the tools).
10+
111
2017-09-15 JF Bastien <[email protected]>
212

313
WTF: use Forward.h when appropriate instead of Vector.h

Source/WebKit/PlatformMac.cmake

+2-2
Original file line numberDiff line numberDiff line change
@@ -543,6 +543,7 @@ set(WebKitLegacyForwardingHeaders
543543
WebCoreStatistics.h
544544
WebDOMOperations.h
545545
WebDOMOperationsPrivate.h
546+
WebDatabaseManagerPrivate.h
546547
WebDataSource.h
547548
WebDataSourcePrivate.h
548549
WebDefaultPolicyDelegate.h
@@ -578,6 +579,7 @@ set(WebKitLegacyForwardingHeaders
578579
WebQuotaManager.h
579580
WebScriptWorld.h
580581
WebSecurityOriginPrivate.h
582+
WebStorageManagerPrivate.h
581583
WebTypesInternal.h
582584
WebUIDelegate.h
583585
WebUIDelegatePrivate.h
@@ -739,8 +741,6 @@ list(APPEND WebKit2_AUTOMATION_PROTOCOL_GENERATOR_EXTRA_FLAGS
739741

740742
# FIXME: These should not be necessary.
741743
file(WRITE ${FORWARDING_HEADERS_DIR}/WebKit/WKImageCG.h "#import <WebKit/Shared/API/c/cg/WKImageCG.h>")
742-
file(WRITE ${FORWARDING_HEADERS_DIR}/WebKit/WebStorageManagerPrivate.h "#import <WebKit/mac/Storage/WebStorageManagerPrivate.h>")
743-
file(WRITE ${FORWARDING_HEADERS_DIR}/WebKit/WebDatabaseManagerPrivate.h "#import <WebKit/mac/Storage/WebDatabaseManagerPrivate.h>")
744744

745745
set(CMAKE_SHARED_LINKER_FLAGS ${CMAKE_SHARED_LINKER_FLAGS} "-compatibility_version 1 -current_version ${WEBKIT_MAC_VERSION}")
746746

Source/WebKitLegacy/ChangeLog

+10
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
2017-09-16 Tim Horton <[email protected]>
2+
3+
Fix the Tools build with CMake on macOS
4+
https://bugs.webkit.org/show_bug.cgi?id=177030
5+
6+
Reviewed by Geoffrey Garen.
7+
8+
* PlatformMac.cmake:
9+
Add some forwarding headers.
10+
111
2017-09-12 Per Arne Vollan <[email protected]>
212

313
[Win] Link errors when building WebKit from .proj files.

Source/WebKitLegacy/PlatformMac.cmake

+3
Original file line numberDiff line numberDiff line change
@@ -325,6 +325,7 @@ set(WebKitLegacy_FORWARDING_HEADERS_DIRECTORIES
325325
mac/Misc
326326
mac/Panels
327327
mac/Plugins
328+
mac/Storage
328329
mac/WebCoreSupport
329330
mac/WebInspector
330331
mac/WebView
@@ -342,6 +343,8 @@ set(WebKitLegacy_FORWARDING_HEADERS_FILES
342343

343344
mac/Plugins/WebBasePluginPackage.h
344345

346+
mac/Storage/WebDatabaseManagerPrivate.h
347+
345348
mac/WebCoreSupport/WebKeyGenerator.h
346349

347350
mac/WebInspector/WebInspector.h

Tools/ChangeLog

+10
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,13 @@
1+
2017-09-16 Tim Horton <[email protected]>
2+
3+
Fix the Tools build with CMake on macOS
4+
https://bugs.webkit.org/show_bug.cgi?id=177030
5+
6+
Reviewed by Geoffrey Garen.
7+
8+
* TestWebKitAPI/CMakeLists.txt:
9+
Add a workaround identical to what the gtest CMakeLists has.
10+
111
2017-09-16 Carlos Garcia Campos <[email protected]>
212

313
[GTK] Bump fontconfig version to 2.12.4

Tools/TestWebKitAPI/CMakeLists.txt

+6
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,12 @@ if (COMPILER_IS_GCC_OR_CLANG)
205205
-Wno-unused-parameter)
206206
endif ()
207207

208+
# FIXME: This works around compatibility problems in the old version of the third-pary
209+
# googletest source code checkout. It should be removed once we upgrade to a newer version.
210+
if (COMPILER_IS_CLANG)
211+
add_definitions(-DGTEST_HAS_TR1_TUPLE=0)
212+
endif ()
213+
208214
if (ENABLE_WEBKIT)
209215
add_library(TestWebKitAPIBase
210216
${test_main_SOURCES}

0 commit comments

Comments
 (0)