Title: [219578] trunk
Revision
219578
Author
mcatanz...@igalia.com
Date
2017-07-17 15:42:06 -0700 (Mon, 17 Jul 2017)

Log Message

[CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
https://bugs.webkit.org/show_bug.cgi?id=174547

Reviewed by Alex Christensen.

.:

* Source/cmake/WebKitMacros.cmake:

Source/_javascript_Core:

* CMakeLists.txt:
* shell/CMakeLists.txt:

Source/ThirdParty:

* brotli/CMakeLists.txt:
* woff2/CMakeLists.txt:
* xdgmime/CMakeLists.txt:

Source/WebCore:

* CMakeLists.txt:
* WebCoreMacros.cmake:

Source/WebKit:

* CMakeLists.txt:

Source/WebKitLegacy:

* CMakeLists.txt:

Tools:

* DumpRenderTree/PlatformWin.cmake:
* DumpRenderTree/TestNetscapePlugIn/CMakeLists.txt:

Modified Paths

Diff

Modified: trunk/ChangeLog (219577 => 219578)


--- trunk/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,12 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * Source/cmake/WebKitMacros.cmake:
+
 2017-07-17  Konstantin Tokarev  <annu...@yandex.ru>
 
         [CMake] Create targets before WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS is called

Modified: trunk/Source/_javascript_Core/CMakeLists.txt (219577 => 219578)


--- trunk/Source/_javascript_Core/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/_javascript_Core/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1234,7 +1234,7 @@
         COMMAND ${PYTHON_EXECUTABLE} ${_generator} ${_input} ${_output}
         VERBATIM)
     list(APPEND _javascript_Core_HEADERS ${_output})
-    ADD_SOURCE_DEPENDENCIES(${_input} ${_output})
+    WEBKIT_ADD_SOURCE_DEPENDENCIES(${_input} ${_output})
 endmacro()
 GENERATE_PYTHON(${CMAKE_CURRENT_SOURCE_DIR}/wasm/generateWasmOpsHeader.py ${CMAKE_CURRENT_SOURCE_DIR}/wasm/generateWasm.py ${CMAKE_CURRENT_SOURCE_DIR}/wasm/wasm.json ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/WasmOps.h)
 GENERATE_PYTHON(${CMAKE_CURRENT_SOURCE_DIR}/wasm/generateWasmValidateInlinesHeader.py ${CMAKE_CURRENT_SOURCE_DIR}/wasm/generateWasm.py ${CMAKE_CURRENT_SOURCE_DIR}/wasm/wasm.json ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/WasmValidateInlines.h)
@@ -1251,7 +1251,7 @@
         COMMAND ${PERL_EXECUTABLE} ${HASH_LUT_GENERATOR} ${_input} > ${_output}
         VERBATIM)
     list(APPEND _javascript_Core_HEADERS ${_output})
-    ADD_SOURCE_DEPENDENCIES(${_input} ${_output})
+    WEBKIT_ADD_SOURCE_DEPENDENCIES(${_input} ${_output})
 endmacro()
 
 # GENERATOR 1-A: LUT creator
@@ -1300,7 +1300,7 @@
     MAIN_DEPENDENCY ${CMAKE_CURRENT_SOURCE_DIR}/create_regex_tables
     COMMAND ${PYTHON_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/create_regex_tables > ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/RegExpJitTables.h
     VERBATIM)
-ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/yarr/YarrPattern.cpp ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/RegExpJitTables.h)
+WEBKIT_ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/yarr/YarrPattern.cpp ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/RegExpJitTables.h)
 
 add_custom_command(
     OUTPUT ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/YarrCanonicalizeUnicode.cpp
@@ -1320,7 +1320,7 @@
     DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/parser/Keywords.table
     COMMAND ${PYTHON_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/KeywordLookupGenerator.py ${CMAKE_CURRENT_SOURCE_DIR}/parser/Keywords.table > ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/KeywordLookup.h
     VERBATIM)
-ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/parser/Lexer.cpp ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/KeywordLookup.h)
+WEBKIT_ADD_SOURCE_DEPENDENCIES(${CMAKE_CURRENT_SOURCE_DIR}/parser/Lexer.cpp ${DERIVED_SOURCES_JAVASCRIPTCORE_DIR}/KeywordLookup.h)
 
 
 # Inspector Interfaces
@@ -1537,7 +1537,7 @@
 
 if (NOT "${PORT}" STREQUAL "Mac")
     if (${_javascript_Core_LIBRARY_TYPE} STREQUAL "SHARED")
-        POPULATE_LIBRARY_VERSION(_javascript_CORE)
+        WEBKIT_POPULATE_LIBRARY_VERSION(_javascript_CORE)
         set_target_properties(_javascript_Core PROPERTIES VERSION ${_javascript_CORE_VERSION} SOVERSION ${_javascript_CORE_VERSION_MAJOR})
         install(TARGETS _javascript_Core DESTINATION "${LIB_INSTALL_DIR}")
     endif ()

Modified: trunk/Source/_javascript_Core/ChangeLog (219577 => 219578)


--- trunk/Source/_javascript_Core/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,13 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+        * shell/CMakeLists.txt:
+
 2017-07-17  Saam Barati  <sbar...@apple.com>
 
         Remove custom defined RELEASE_ASSERT in DFGObjectAllocationSinkingPhase

Modified: trunk/Source/_javascript_Core/shell/CMakeLists.txt (219577 => 219578)


--- trunk/Source/_javascript_Core/shell/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/_javascript_Core/shell/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -44,7 +44,7 @@
 endif ()
 
 if (JSC_LINK_FLAGS)
-    ADD_TARGET_PROPERTIES(jsc LINK_FLAGS "${JSC_LINK_FLAGS}")
+    WEBKIT_ADD_TARGET_PROPERTIES(jsc LINK_FLAGS "${JSC_LINK_FLAGS}")
 endif ()
 
 if (SHOULD_INSTALL_JS_SHELL)

Modified: trunk/Source/ThirdParty/ChangeLog (219577 => 219578)


--- trunk/Source/ThirdParty/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/ThirdParty/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,14 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * brotli/CMakeLists.txt:
+        * woff2/CMakeLists.txt:
+        * xdgmime/CMakeLists.txt:
+
 2017-07-13  Michael Catanzaro  <mcatanz...@igalia.com>
 
         Fix compiler warnings when building with GCC 7

Modified: trunk/Source/ThirdParty/brotli/CMakeLists.txt (219577 => 219578)


--- trunk/Source/ThirdParty/brotli/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/ThirdParty/brotli/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -18,5 +18,5 @@
 WEBKIT_SET_EXTRA_COMPILER_FLAGS(brotli)
 
 if (COMPILER_IS_GCC_OR_CLANG)
-    ADD_TARGET_PROPERTIES(brotli COMPILE_FLAGS "-Wno-cast-align -Wno-implicit-fallthrough")
+    WEBKIT_ADD_TARGET_PROPERTIES(brotli COMPILE_FLAGS "-Wno-cast-align -Wno-implicit-fallthrough")
 endif ()

Modified: trunk/Source/ThirdParty/woff2/CMakeLists.txt (219577 => 219578)


--- trunk/Source/ThirdParty/woff2/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/ThirdParty/woff2/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -19,9 +19,9 @@
 WEBKIT_SET_EXTRA_COMPILER_FLAGS(woff2)
 
 if (COMPILER_IS_GCC_OR_CLANG)
-    ADD_TARGET_PROPERTIES(woff2 COMPILE_FLAGS "-Wno-sign-compare -Wno-unused-variable -Wno-unused-parameter")
+    WEBKIT_ADD_TARGET_PROPERTIES(woff2 COMPILE_FLAGS "-Wno-sign-compare -Wno-unused-variable -Wno-unused-parameter")
 endif ()
 
 if (CMAKE_COMPILER_IS_GNUCXX)
-    ADD_TARGET_PROPERTIES(woff2 COMPILE_FLAGS "-Wno-unused-but-set-variable")
+    WEBKIT_ADD_TARGET_PROPERTIES(woff2 COMPILE_FLAGS "-Wno-unused-but-set-variable")
 endif ()

Modified: trunk/Source/ThirdParty/xdgmime/CMakeLists.txt (219577 => 219578)


--- trunk/Source/ThirdParty/xdgmime/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/ThirdParty/xdgmime/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -22,5 +22,5 @@
 WEBKIT_SET_EXTRA_COMPILER_FLAGS(xdgmime)
 
 if (COMPILER_IS_GCC_OR_CLANG)
-    ADD_TARGET_PROPERTIES(xdgmime COMPILE_FLAGS "-Wno-sign-compare -Wno-unused-parameter")
+    WEBKIT_ADD_TARGET_PROPERTIES(xdgmime COMPILE_FLAGS "-Wno-sign-compare -Wno-unused-parameter")
 endif ()

Modified: trunk/Source/WebCore/CMakeLists.txt (219577 => 219578)


--- trunk/Source/WebCore/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebCore/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -3935,8 +3935,8 @@
 WEBKIT_WRAP_SOURCELIST(${WebCoreTestSupport_IDL_FILES} ${WebCoreTestSupport_SOURCES})
 
 if (MSVC)
-    ADD_PRECOMPILED_HEADER("WebCorePrefix.h" "WebCorePrefix.cpp" WebCore_SOURCES)
-    ADD_PRECOMPILED_HEADER("WebCorePrefix.h" "WebCoreDerivedSourcesPrefix.cpp" WebCore_DERIVED_SOURCES)
+    WEBKIT_ADD_PRECOMPILED_HEADER("WebCorePrefix.h" "WebCorePrefix.cpp" WebCore_SOURCES)
+    WEBKIT_ADD_PRECOMPILED_HEADER("WebCorePrefix.h" "WebCoreDerivedSourcesPrefix.cpp" WebCore_DERIVED_SOURCES)
 endif ()
 
 # This is split into a separate library as a workaround for command line length
@@ -3968,11 +3968,11 @@
 # See https://bugs.webkit.org/show_bug.cgi?id=146440
 string(TOLOWER ${CMAKE_HOST_SYSTEM_PROCESSOR} LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR)
 if (CMAKE_COMPILER_IS_GNUCXX AND "${LOWERCASE_CMAKE_HOST_SYSTEM_PROCESSOR}" MATCHES "(i[3-6]86|x86)$")
-    ADD_TARGET_PROPERTIES(WebCore COMPILE_FLAGS "-fno-tree-sra")
+    WEBKIT_ADD_TARGET_PROPERTIES(WebCore COMPILE_FLAGS "-fno-tree-sra")
 endif ()
 
 if (MSVC)
-    ADD_PRECOMPILED_HEADER("WebCoreTestSupportPrefix.h" "testing/js/WebCoreTestSupportPrefix.cpp" WebCoreTestSupport_SOURCES)
+    WEBKIT_ADD_PRECOMPILED_HEADER("WebCoreTestSupportPrefix.h" "testing/js/WebCoreTestSupportPrefix.cpp" WebCoreTestSupport_SOURCES)
 endif ()
 
 add_library(WebCoreTestSupport ${WebCoreTestSupport_LIBRARY_TYPE} ${WebCoreTestSupport_SOURCES})
@@ -3992,7 +3992,7 @@
 
     # Suppress null conversion warnings for sources in Source/ThirdParty/ANGLE
     if (COMPILER_IS_CLANG)
-        ADD_TARGET_PROPERTIES(ANGLESupport COMPILE_FLAGS "-Wno-null-conversion")
+        WEBKIT_ADD_TARGET_PROPERTIES(ANGLESupport COMPILE_FLAGS "-Wno-null-conversion")
     endif ()
 
     # Enable the ESSL and GLSL translators.

Modified: trunk/Source/WebCore/ChangeLog (219577 => 219578)


--- trunk/Source/WebCore/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebCore/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,13 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+        * WebCoreMacros.cmake:
+
 2017-07-17  Alex Christensen  <achristen...@webkit.org>
 
         Modernize content extension code

Modified: trunk/Source/WebCore/WebCoreMacros.cmake (219577 => 219578)


--- trunk/Source/WebCore/WebCoreMacros.cmake	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebCore/WebCoreMacros.cmake	2017-07-17 22:42:06 UTC (rev 219578)
@@ -58,7 +58,7 @@
         list(APPEND _tmp "${DERIVED_SOURCES_WEBCORE_DIR}/${f}")
     endforeach ()
 
-    ADD_SOURCE_DEPENDENCIES(${_source} ${_tmp})
+    WEBKIT_ADD_SOURCE_DEPENDENCIES(${_source} ${_tmp})
     unset(_tmp)
 endmacro()
 
@@ -69,7 +69,7 @@
         DEPENDS ${_javascript_Core_SCRIPTS_DIR}/make-js-file-arrays.py ${${_scripts}}
         COMMAND ${PYTHON_EXECUTABLE} ${_javascript_Core_SCRIPTS_DIR}/make-js-file-arrays.py -n ${_namespace} ${_output_h} ${_output_cpp} ${${_scripts}}
         VERBATIM)
-    ADD_SOURCE_DEPENDENCIES(${${_scripts_dependencies}} ${_output_h} ${_output_cpp})
+    WEBKIT_ADD_SOURCE_DEPENDENCIES(${${_scripts_dependencies}} ${_output_h} ${_output_cpp})
 endmacro()
 
 

Modified: trunk/Source/WebKit/CMakeLists.txt (219577 => 219578)


--- trunk/Source/WebKit/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebKit/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -844,11 +844,11 @@
 
 # Suppress unused parameter warnings for sources in WebKit2.
 if (COMPILER_IS_GCC_OR_CLANG)
-    ADD_TARGET_PROPERTIES(WebKit2 COMPILE_FLAGS "-Wno-unused-parameter")
+    WEBKIT_ADD_TARGET_PROPERTIES(WebKit2 COMPILE_FLAGS "-Wno-unused-parameter")
 endif ()
 
 if (WebKit2_VERSION_SCRIPT)
-    ADD_TARGET_PROPERTIES(WebKit2 LINK_FLAGS "${WebKit2_VERSION_SCRIPT}")
+    WEBKIT_ADD_TARGET_PROPERTIES(WebKit2 LINK_FLAGS "${WebKit2_VERSION_SCRIPT}")
 endif ()
 
 add_executable(WebProcess ${WebProcess_SOURCES})
@@ -919,7 +919,7 @@
 
     set_target_properties(NetworkProcess PROPERTIES RUNTIME_OUTPUT_DIRECTORY "${_net_xpc_dir}/MacOS")
 else ()
-    POPULATE_LIBRARY_VERSION(WEBKIT2)
+    WEBKIT_POPULATE_LIBRARY_VERSION(WEBKIT2)
     set_target_properties(WebKit2 PROPERTIES VERSION ${WEBKIT2_VERSION} SOVERSION ${WEBKIT2_VERSION_MAJOR})
 
     install(TARGETS WebKit2 WebProcess NetworkProcess

Modified: trunk/Source/WebKit/ChangeLog (219577 => 219578)


--- trunk/Source/WebKit/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebKit/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,12 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+
 2017-07-17  Alex Christensen  <achristen...@webkit.org>
 
         Modernize content extension code

Modified: trunk/Source/WebKitLegacy/CMakeLists.txt (219577 => 219578)


--- trunk/Source/WebKitLegacy/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebKitLegacy/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -39,7 +39,7 @@
 WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS()
 
 if (MSVC)
-    ADD_PRECOMPILED_HEADER("WebKitPrefix.h" "win/WebKitPrefix.cpp" WebKit_SOURCES)
+    WEBKIT_ADD_PRECOMPILED_HEADER("WebKitPrefix.h" "win/WebKitPrefix.cpp" WebKit_SOURCES)
     list(APPEND WebKit_SOURCES win/WebKit.resources/WebKit.rc)
     add_definitions(/DWEBKIT_EXPORTS /DFRAMEWORK_NAME=WebKit)
 endif ()

Modified: trunk/Source/WebKitLegacy/ChangeLog (219577 => 219578)


--- trunk/Source/WebKitLegacy/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/WebKitLegacy/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,12 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * CMakeLists.txt:
+
 2017-07-17  Konstantin Tokarev  <annu...@yandex.ru>
 
         [CMake] Create targets before WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS is called

Modified: trunk/Source/cmake/WebKitMacros.cmake (219577 => 219578)


--- trunk/Source/cmake/WebKitMacros.cmake	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Source/cmake/WebKitMacros.cmake	2017-07-17 22:42:06 UTC (rev 219578)
@@ -17,7 +17,7 @@
 endmacro()
 
 # Append the given dependencies to the source file
-macro(ADD_SOURCE_DEPENDENCIES _source _deps)
+macro(WEBKIT_ADD_SOURCE_DEPENDENCIES _source _deps)
     set(_tmp)
     get_source_file_property(_tmp ${_source} OBJECT_DEPENDS)
     if (NOT _tmp)
@@ -32,7 +32,7 @@
     unset(_tmp)
 endmacro()
 
-macro(ADD_PRECOMPILED_HEADER _header _cpp _source)
+macro(WEBKIT_ADD_PRECOMPILED_HEADER _header _cpp _source)
     if (MSVC)
         get_filename_component(PrecompiledBasename ${_cpp} NAME_WE)
         file(MAKE_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}/${_source}")
@@ -53,7 +53,7 @@
             PROPERTIES COMPILE_FLAGS "/Yu\"${_header}\" /FI\"${_header}\" /Fp\"${PrecompiledBinary}\"")
 
         foreach (_src ${_sources})
-            ADD_SOURCE_DEPENDENCIES(${_src} ${PrecompiledBinary})
+            WEBKIT_ADD_SOURCE_DEPENDENCIES(${_src} ${PrecompiledBinary})
         endforeach ()
 
         list(APPEND ${_source} ${_cpp})
@@ -222,7 +222,7 @@
 
 # Append the given flag to the target property.
 # Builds on top of get_target_property() and set_target_properties()
-macro(ADD_TARGET_PROPERTIES _target _property _flags)
+macro(WEBKIT_ADD_TARGET_PROPERTIES _target _property _flags)
     get_target_property(_tmp ${_target} ${_property})
     if (NOT _tmp)
         set(_tmp "")
@@ -234,9 +234,9 @@
 
     set_target_properties(${_target} PROPERTIES ${_property} ${_tmp})
     unset(_tmp)
-endmacro(ADD_TARGET_PROPERTIES _target _property _flags)
+endmacro()
 
-macro(POPULATE_LIBRARY_VERSION library_name)
+macro(WEBKIT_POPULATE_LIBRARY_VERSION library_name)
     if (NOT DEFINED ${library_name}_VERSION_MAJOR)
         set(${library_name}_VERSION_MAJOR ${PROJECT_VERSION_MAJOR})
     endif ()

Modified: trunk/Tools/ChangeLog (219577 => 219578)


--- trunk/Tools/ChangeLog	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Tools/ChangeLog	2017-07-17 22:42:06 UTC (rev 219578)
@@ -1,3 +1,13 @@
+2017-07-17  Michael Catanzaro  <mcatanz...@igalia.com>
+
+        [CMake] Macros in WebKitMacros.cmake should be prefixed with WEBKIT_ namespace
+        https://bugs.webkit.org/show_bug.cgi?id=174547
+
+        Reviewed by Alex Christensen.
+
+        * DumpRenderTree/PlatformWin.cmake:
+        * DumpRenderTree/TestNetscapePlugIn/CMakeLists.txt:
+
 2017-07-17  Yoshiaki Jitsukawa  <yoshiaki.jitsuk...@sony.com>
 
         Update style checker to deal with "final"

Modified: trunk/Tools/DumpRenderTree/PlatformWin.cmake (219577 => 219578)


--- trunk/Tools/DumpRenderTree/PlatformWin.cmake	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Tools/DumpRenderTree/PlatformWin.cmake	2017-07-17 22:42:06 UTC (rev 219578)
@@ -104,7 +104,7 @@
     )
 endif ()
 
-ADD_PRECOMPILED_HEADER("DumpRenderTreePrefix.h" "win/DumpRenderTreePrefix.cpp" DumpRenderTreeLib_SOURCES)
+WEBKIT_ADD_PRECOMPILED_HEADER("DumpRenderTreePrefix.h" "win/DumpRenderTreePrefix.cpp" DumpRenderTreeLib_SOURCES)
 set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} /NODEFAULTLIB:MSVCRT /NODEFAULTLIB:MSVCRTD")
 add_definitions(-DUSE_CONSOLE_ENTRY_POINT)
 

Modified: trunk/Tools/DumpRenderTree/TestNetscapePlugIn/CMakeLists.txt (219577 => 219578)


--- trunk/Tools/DumpRenderTree/TestNetscapePlugIn/CMakeLists.txt	2017-07-17 22:40:18 UTC (rev 219577)
+++ trunk/Tools/DumpRenderTree/TestNetscapePlugIn/CMakeLists.txt	2017-07-17 22:42:06 UTC (rev 219578)
@@ -58,4 +58,4 @@
 WEBKIT_SET_EXTRA_COMPILER_FLAGS(TestNetscapePlugIn)
 
 # Suppress unused parameter warnings for sources in WebKit2.
-ADD_TARGET_PROPERTIES(TestNetscapePlugIn COMPILE_FLAGS "-Wno-unused-parameter")
+WEBKIT_ADD_TARGET_PROPERTIES(TestNetscapePlugIn COMPILE_FLAGS "-Wno-unused-parameter")
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to