Attachment 368332 Details for Bug 197206 – Patch

[patch] Patch

bug-197206.diff (text/plain), 16.03 KB, created by Don Olmstead on 2019-04-26 10:35:48 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Don Olmstead

Created: 2019-04-26 10:35:48 PDT

Size: 16.03 KB

patch

obsolete

>diff --git a/ChangeLog b/ChangeLog
>index 1f0e867939f..91a5e4272cd 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,3 +1,21 @@
>+2019-04-26  Don Olmstead  <don.olmstead@sony.com>
>+
>+        [CMake] Add WEBKIT_EXECUTABLE macro
>+        https://bugs.webkit.org/show_bug.cgi?id=197206
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add WEBKIT_EXECUTABLE macro for creating executable targets. This wraps the creation of
>+        executable targets using the same conventions of WEBKIT_FRAMEWORK.
>+        
>+        Common code from the two macros were moved to WEBKIT_TARGET which is meant as an internal
>+        macro.
>+
>+        Add WEBKIT_WRAP_EXECUTABLE macro to support Windows targets which use a DLLLauncherMain to
>+        launch the executable.
>+
>+        * Source/cmake/WebKitMacros.cmake:
>+
> 2019-04-25  Fujii Hironori  <Hironori.Fujii@sony.com>
> 
>         Unreviewed, rolling out r244669.
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 6114aa50fea..bd2f0df4437 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,21 @@
>+2019-04-26  Don Olmstead  <don.olmstead@sony.com>
>+
>+        [CMake] Add WEBKIT_EXECUTABLE macro
>+        https://bugs.webkit.org/show_bug.cgi?id=197206
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Migrate to WEBKIT_EXECUTABLE for the jsc and test targets.
>+
>+        * b3/air/testair.cpp:
>+        * b3/testb3.cpp:
>+        * dfg/testdfg.cpp:
>+        * shell/CMakeLists.txt:
>+        * shell/PlatformGTK.cmake:
>+        * shell/PlatformMac.cmake:
>+        * shell/PlatformPlayStation.cmake:
>+        * shell/PlatformWin.cmake:
>+
> 2019-04-26  Don Olmstead  <don.olmstead@sony.com>
> 
>         Add WTF::findIgnoringASCIICaseWithoutLength to replace strcasestr
>diff --git a/Source/JavaScriptCore/b3/air/testair.cpp b/Source/JavaScriptCore/b3/air/testair.cpp
>index 9dc9966a944..ef1544c07bb 100644
>--- a/Source/JavaScriptCore/b3/air/testair.cpp
>+++ b/Source/JavaScriptCore/b3/air/testair.cpp
>@@ -2216,3 +2216,10 @@ int main(int argc, char** argv)
> 
>     return 0;
> }
>+
>+#if OS(WINDOWS)
>+extern "C" __declspec(dllexport) int WINAPI dllLauncherEntryPoint(int argc, const char* argv[])
>+{
>+    return main(argc, const_cast<char**>(argv));
>+}
>+#endif
>diff --git a/Source/JavaScriptCore/b3/testb3.cpp b/Source/JavaScriptCore/b3/testb3.cpp
>index ff3b3b53831..7f31679b413 100644
>--- a/Source/JavaScriptCore/b3/testb3.cpp
>+++ b/Source/JavaScriptCore/b3/testb3.cpp
>@@ -18800,3 +18800,9 @@ int main(int argc, char** argv)
>     return 0;
> }
> 
>+#if OS(WINDOWS)
>+extern "C" __declspec(dllexport) int WINAPI dllLauncherEntryPoint(int argc, const char* argv[])
>+{
>+    return main(argc, const_cast<char**>(argv));
>+}
>+#endif
>diff --git a/Source/JavaScriptCore/dfg/testdfg.cpp b/Source/JavaScriptCore/dfg/testdfg.cpp
>index 5335e64661b..6d3322910ad 100644
>--- a/Source/JavaScriptCore/dfg/testdfg.cpp
>+++ b/Source/JavaScriptCore/dfg/testdfg.cpp
>@@ -118,3 +118,10 @@ int main(int argc, char** argv)
> 
>     return 0;
> }
>+
>+#if OS(WINDOWS)
>+extern "C" __declspec(dllexport) int WINAPI dllLauncherEntryPoint(int argc, const char* argv[])
>+{
>+    return main(argc, const_cast<char**>(argv));
>+}
>+#endif
>diff --git a/Source/JavaScriptCore/shell/CMakeLists.txt b/Source/JavaScriptCore/shell/CMakeLists.txt
>index aa7ce64ec1c..1ddebce3436 100644
>--- a/Source/JavaScriptCore/shell/CMakeLists.txt
>+++ b/Source/JavaScriptCore/shell/CMakeLists.txt
>@@ -1,22 +1,29 @@
>-set(JSC_SOURCES
>+set(jsc_SOURCES
>     ../jsc.cpp
> )
> 
>-set(JSC_LIBRARIES
>+set(jsc_LIBRARIES
>     ${CMAKE_DL_LIBS}
>     JavaScriptCore${DEBUG_SUFFIX}
>     WTF${DEBUG_SUFFIX}
> )
> 
>+set(jsc_PRIVATE_INCLUDE_DIRECTORIES
>+    ${JavaScriptCore_INCLUDE_DIRECTORIES}
>+    ${JavaScriptCore_PRIVATE_INCLUDE_DIRECTORIES}
>+    ${JavaScriptCore_PRIVATE_FRAMEWORK_HEADERS_DIR}
>+    ${JavaScriptCore_SYSTEM_INCLUDE_DIRECTORIES}
>+)
>+
> if (WTF_OS_MAC_OS_X)
>-    list(APPEND JSC_LIBRARIES edit)
>+    list(APPEND jsc_LIBRARIES edit)
> endif ()
> 
> if ("${JavaScriptCore_LIBRARY_TYPE}" MATCHES "STATIC")
>     add_definitions(-DSTATICALLY_LINKED_WITH_JavaScriptCore)
> endif ()
> 
>-set(TESTAPI_SOURCES
>+set(testapi_SOURCES
>     ../API/tests/CompareAndSwapTest.cpp
>     ../API/tests/CustomGlobalObjectClassTest.c
>     ../API/tests/ExecutionTimeLimitTest.cpp
>@@ -30,72 +37,65 @@ set(TESTAPI_SOURCES
>     ../API/tests/testapi.c
>     ../API/tests/testapi.cpp
> )
>+set(testapi_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testapi_LIBRARIES JavaScriptCore)
> 
>-WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS()
>-
>-WEBKIT_WRAP_SOURCELIST(${JSC_SOURCES})
>-WEBKIT_WRAP_SOURCELIST(${TESTAPI_SOURCES})
>-include_directories(./ ${JavaScriptCore_INCLUDE_DIRECTORIES} ${JavaScriptCore_PRIVATE_INCLUDE_DIRECTORIES})
>-include_directories(SYSTEM ${JavaScriptCore_SYSTEM_INCLUDE_DIRECTORIES})
>-add_executable(jsc ${JSC_SOURCES})
>-target_link_libraries(jsc ${JSC_LIBRARIES})
>-
>-if (JSC_OUTPUT_NAME)
>-    set_target_properties(jsc PROPERTIES OUTPUT_NAME ${JSC_OUTPUT_NAME})
>-endif ()
>-
>-if (JSC_LINK_FLAGS)
>-    WEBKIT_ADD_TARGET_PROPERTIES(jsc LINK_FLAGS "${JSC_LINK_FLAGS}")
>-endif ()
>-
>-if (SHOULD_INSTALL_JS_SHELL)
>-    install(TARGETS jsc DESTINATION "${LIBEXEC_INSTALL_DIR}")
>-endif ()
>-
>-if (TARGET jscLib)
>-    add_dependencies(jsc jscLib)
>-endif ()
>-
>-set(TESTMASM_SOURCES
>-    ../assembler/testmasm.cpp
>-)
>-
>-set(TESTB3_SOURCES
>-    ../b3/testb3.cpp
>-)
>+set(testmasm_SOURCES ../assembler/testmasm.cpp)
>+set(testmasm_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testmasm_LIBRARIES JavaScriptCore)
> 
>-set(TESTAIR_SOURCES
>-    ../b3/air/testair.cpp
>-)
>+set(testRegExp_SOURCES ../testRegExp.cpp)
>+set(testRegExp_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testRegExp_LIBRARIES JavaScriptCore)
> 
>-set(TESTDFG_SOURCES
>-    ../dfg/testdfg.cpp
>-)
>+set(testb3_SOURCES ../b3/testb3.cpp)
>+set(testb3_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testb3_LIBRARIES JavaScriptCore)
> 
>-if (DEVELOPER_MODE AND NOT WIN32)
>-    add_executable(testmasm ${TESTMASM_SOURCES})
>-    target_link_libraries(testmasm ${JSC_LIBRARIES})
>+set(testair_SOURCES ../b3/air/testair.cpp)
>+set(testair_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testair_LIBRARIES JavaScriptCore)
> 
>-    add_executable(testb3 ${TESTB3_SOURCES})
>-    target_link_libraries(testb3 ${JSC_LIBRARIES})
>+set(testdfg_SOURCES ../dfg/testdfg.cpp)
>+set(testdfg_PRIVATE_INCLUDE_DIRECTORIES ${jsc_PRIVATE_INCLUDE_DIRECTORIES})
>+set(testdfg_LIBRARIES JavaScriptCore)
> 
>-    add_executable(testair ${TESTAIR_SOURCES})
>-    target_link_libraries(testair ${JSC_LIBRARIES})
>+WEBKIT_EXECUTABLE_DECLARE(jsc)
>+WEBKIT_EXECUTABLE_DECLARE(testapi)
>+WEBKIT_EXECUTABLE_DECLARE(testRegExp)
>+WEBKIT_EXECUTABLE_DECLARE(testmasm)
>+WEBKIT_EXECUTABLE_DECLARE(testb3)
>+WEBKIT_EXECUTABLE_DECLARE(testair)
>+WEBKIT_EXECUTABLE_DECLARE(testdfg)
> 
>-    add_executable(testdfg ${TESTDFG_SOURCES})
>-    target_link_libraries(testdfg ${JSC_LIBRARIES})
>+WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS()
> 
>-    add_executable(testapi ${TESTAPI_SOURCES})
>-    target_link_libraries(testapi ${JSC_LIBRARIES})
>+if (DEVELOPER_MODE)
>+    WEBKIT_EXECUTABLE(jsc)
>+
>+    if (jsc_LINK_FLAGS)
>+        WEBKIT_ADD_TARGET_PROPERTIES(jsc LINK_FLAGS "${JSC_LINK_FLAGS}")
>+    endif ()
>+
>+    if (SHOULD_INSTALL_JS_SHELL)
>+        install(TARGETS jsc DESTINATION "${LIBEXEC_INSTALL_DIR}")
>+    endif ()
>+
>+    WEBKIT_EXECUTABLE(testapi)
>+    WEBKIT_EXECUTABLE(testRegExp)
>+    WEBKIT_EXECUTABLE(testmasm)
>+    WEBKIT_EXECUTABLE(testb3)
>+    WEBKIT_EXECUTABLE(testair)
>+    WEBKIT_EXECUTABLE(testdfg)
>+
>+    file(COPY
>+        "${JAVASCRIPTCORE_DIR}/API/tests/testapi.js"
>+        DESTINATION
>+        ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/testapiScripts
>+    )
>+    file(COPY
>+        "${JAVASCRIPTCORE_DIR}/API/tests/testapi-function-overrides.js"
>+        DESTINATION
>+        ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/testapiScripts
>+    )
> endif ()
>-
>-file(COPY
>-    "${JAVASCRIPTCORE_DIR}/API/tests/testapi.js"
>-    DESTINATION
>-    ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/testapiScripts
>-)
>-file(COPY
>-    "${JAVASCRIPTCORE_DIR}/API/tests/testapi-function-overrides.js"
>-    DESTINATION
>-    ${CMAKE_RUNTIME_OUTPUT_DIRECTORY}/testapiScripts
>-)
>diff --git a/Source/JavaScriptCore/shell/PlatformGTK.cmake b/Source/JavaScriptCore/shell/PlatformGTK.cmake
>index 623183a2b08..c4ac7c9a209 100644
>--- a/Source/JavaScriptCore/shell/PlatformGTK.cmake
>+++ b/Source/JavaScriptCore/shell/PlatformGTK.cmake
>@@ -1,3 +1,3 @@
>-list(APPEND JSC_LIBRARIES
>+list(APPEND jsc_LIBRARIES
>     ${GLIB_LIBRARIES}
> )
>diff --git a/Source/JavaScriptCore/shell/PlatformMac.cmake b/Source/JavaScriptCore/shell/PlatformMac.cmake
>index 7c0fea252e6..830680b24f7 100644
>--- a/Source/JavaScriptCore/shell/PlatformMac.cmake
>+++ b/Source/JavaScriptCore/shell/PlatformMac.cmake
>@@ -1,4 +1,4 @@
>-set(TESTAPI_OBJC_SOURCES
>+set(testapi_OBJC_SOURCES
>     ../API/tests/CurrentThisInsideBlockGetterTest.mm
>     ../API/tests/DateTests.mm
>     ../API/tests/JSExportTests.mm
>@@ -6,6 +6,5 @@ set(TESTAPI_OBJC_SOURCES
>     ../API/tests/Regress141809.mm
>     ../API/tests/testapi.mm
> )
>-list(APPEND TESTAPI_SOURCES ${TESTAPI_OBJC_SOURCES})
>-
>-set_source_files_properties(${TESTAPI_OBJC_SOURCES} PROPERTIES COMPILE_FLAGS -fobjc-arc)
>+list(APPEND testapi_SOURCES ${testapi_OBJC_SOURCES})
>+set_source_files_properties(${testapi_OBJC_SOURCES} PROPERTIES COMPILE_FLAGS -fobjc-arc)
>diff --git a/Source/JavaScriptCore/shell/PlatformPlayStation.cmake b/Source/JavaScriptCore/shell/PlatformPlayStation.cmake
>index 0937f931d0e..56b43d83034 100644
>--- a/Source/JavaScriptCore/shell/PlatformPlayStation.cmake
>+++ b/Source/JavaScriptCore/shell/PlatformPlayStation.cmake
>@@ -1,5 +1,4 @@
>-
>-list(APPEND JSC_SOURCES
>+list(APPEND jsc_SOURCES
>     ${JAVASCRIPTCORE_DIR}/shell/playstation/Initializer.cpp
> )
> 
>@@ -8,4 +7,4 @@ find_library(LIBTESTWRAPPERS testwrappers PATHS ${WEBKIT_LIBRARIES_DIR}/lib)
> set(PLAYSTATION_jsc_PROCESS_NAME "JSCShell")
> set(PLAYSTATION_jsc_MAIN_THREAD_NAME "JSCShell")
> set(PLAYSTATION_jsc_WRAP fopen getcwd main)
>-list(APPEND JSC_LIBRARIES ${LIBTESTWRAPPERS})
>+list(APPEND jsc_LIBRARIES ${LIBTESTWRAPPERS})
>diff --git a/Source/JavaScriptCore/shell/PlatformWin.cmake b/Source/JavaScriptCore/shell/PlatformWin.cmake
>index aafc629fb63..817c82cd55c 100644
>--- a/Source/JavaScriptCore/shell/PlatformWin.cmake
>+++ b/Source/JavaScriptCore/shell/PlatformWin.cmake
>@@ -1,40 +1,46 @@
>-include_directories(./ PRIVATE ${JavaScriptCore_INCLUDE_DIRECTORIES} ${JavaScriptCore_PRIVATE_INCLUDE_DIRECTORIES} ${JavaScriptCore_PRIVATE_FRAMEWORK_HEADERS_DIR})
>-include_directories(SYSTEM ${JavaScriptCore_SYSTEM_INCLUDE_DIRECTORIES})
>-add_library(jscLib SHARED ${JSC_SOURCES})
>-
>-list(APPEND JSC_LIBRARIES
>-    Winmm
>-)
>-
>-target_link_libraries(jscLib ${JSC_LIBRARIES})
>-
> if (${WTF_PLATFORM_WIN_CAIRO})
>     add_definitions(-DWIN_CAIRO)
> endif ()
>-
>-set(JSC_SOURCES ${JAVASCRIPTCORE_DIR}/shell/DLLLauncherMain.cpp)
>-set(JSC_LIBRARIES shlwapi)
> add_definitions(-DUSE_CONSOLE_ENTRY_POINT)
> 
>-set(JSC_OUTPUT_NAME "jsc${DEBUG_SUFFIX}")
>+WEBKIT_WRAP_EXECUTABLE(jsc
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(jsc_OUTPUT_NAME jsc${DEBUG_SUFFIX})
> 
>-add_library(testRegExpLib SHARED ../testRegExp.cpp)
>-add_executable(testRegExp ${JSC_SOURCES})
>-set_target_properties(testRegExp PROPERTIES OUTPUT_NAME "testRegExp${DEBUG_SUFFIX}")
>-target_link_libraries(testRegExp shlwapi)
>-add_dependencies(testRegExp testRegExpLib)
>-target_link_libraries(testRegExpLib JavaScriptCore)
>+WEBKIT_WRAP_EXECUTABLE(testapi
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testapi_OUTPUT_NAME testapi${DEBUG_SUFFIX})
> 
>-add_library(testapiLib SHARED ${TESTAPI_SOURCES})
>-add_executable(testapi ${JSC_SOURCES})
>-set_target_properties(testapi PROPERTIES OUTPUT_NAME "testapi${DEBUG_SUFFIX}")
>-target_link_libraries(testapi shlwapi)
>-add_dependencies(testapi testapiLib)
>-target_link_libraries(testapiLib JavaScriptCore)
>+WEBKIT_WRAP_EXECUTABLE(testRegExp
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testRegExp_OUTPUT_NAME testRegExp${DEBUG_SUFFIX})
> 
>-add_library(testmasmLib SHARED ../assembler/testmasm.cpp)
>-add_executable(testmasm ${JSC_SOURCES})
>-set_target_properties(testmasm PROPERTIES OUTPUT_NAME "testmasm${DEBUG_SUFFIX}")
>-target_link_libraries(testmasm shlwapi)
>-add_dependencies(testmasm testmasmLib)
>-target_link_libraries(testmasmLib JavaScriptCore)
>+WEBKIT_WRAP_EXECUTABLE(testmasm
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testmasm_OUTPUT_NAME testmasm${DEBUG_SUFFIX})
>+
>+WEBKIT_WRAP_EXECUTABLE(testb3
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testb3_OUTPUT_NAME testb3${DEBUG_SUFFIX})
>+
>+WEBKIT_WRAP_EXECUTABLE(testair
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testair_OUTPUT_NAME testair${DEBUG_SUFFIX})
>+
>+WEBKIT_WRAP_EXECUTABLE(testdfg
>+    SOURCES DLLLauncherMain.cpp
>+    LIBRARIES shlwapi
>+)
>+set(testdfg_OUTPUT_NAME testdfg${DEBUG_SUFFIX})
>diff --git a/Source/cmake/WebKitMacros.cmake b/Source/cmake/WebKitMacros.cmake
>index a065b1cdc22..324bce3269e 100644
>--- a/Source/cmake/WebKitMacros.cmake
>+++ b/Source/cmake/WebKitMacros.cmake
>@@ -142,16 +142,28 @@ macro(WEBKIT_FRAMEWORK_DECLARE _target)
>     add_library(${_target} ${${_target}_LIBRARY_TYPE} "${CMAKE_BINARY_DIR}/cmakeconfig.h")
> endmacro()
> 
>-macro(WEBKIT_FRAMEWORK _target)
>-    target_sources(${_target} PRIVATE
>+macro(WEBKIT_EXECUTABLE_DECLARE _target)
>+    add_executable(${_target} "${CMAKE_BINARY_DIR}/cmakeconfig.h")
>+endmacro()
>+
>+macro(WEBKIT_TARGET _target _target_name)
>+    target_sources(${_target_name} PRIVATE
>         ${${_target}_HEADERS}
>         ${${_target}_SOURCES}
>     )
>-    target_include_directories(${_target} PUBLIC "$<BUILD_INTERFACE:${${_target}_INCLUDE_DIRECTORIES}>")
>-    target_include_directories(${_target} SYSTEM PRIVATE "$<BUILD_INTERFACE:${${_target}_SYSTEM_INCLUDE_DIRECTORIES}>")
>-    target_include_directories(${_target} PRIVATE "$<BUILD_INTERFACE:${${_target}_PRIVATE_INCLUDE_DIRECTORIES}>")
>-    target_link_libraries(${_target} ${${_target}_LIBRARIES})
>-    set_target_properties(${_target} PROPERTIES COMPILE_DEFINITIONS "BUILDING_${_target}")
>+    target_include_directories(${_target_name} PUBLIC "$<BUILD_INTERFACE:${${_target}_INCLUDE_DIRECTORIES}>")
>+    target_include_directories(${_target_name} SYSTEM PRIVATE "$<BUILD_INTERFACE:${${_target}_SYSTEM_INCLUDE_DIRECTORIES}>")
>+    target_include_directories(${_target_name} PRIVATE "$<BUILD_INTERFACE:${${_target}_PRIVATE_INCLUDE_DIRECTORIES}>")
>+    target_link_libraries(${_target_name} ${${_target}_LIBRARIES})
>+    set_target_properties(${_target_name} PROPERTIES COMPILE_DEFINITIONS "BUILDING_${_target}")
>+
>+    if (${_target}_DEPENDENCIES)
>+        add_dependencies(${_target_name} ${${_target}_DEPENDENCIES})
>+    endif ()
>+endmacro()
>+
>+macro(WEBKIT_FRAMEWORK _target)
>+    WEBKIT_TARGET(${_target} ${_target})
> 
>     if (${_target}_OUTPUT_NAME)
>         set_target_properties(${_target} PROPERTIES OUTPUT_NAME ${${_target}_OUTPUT_NAME})
>@@ -172,6 +184,41 @@ macro(WEBKIT_FRAMEWORK _target)
>     endif ()
> endmacro()
> 
>+macro(WEBKIT_EXECUTABLE _target)
>+    WEBKIT_TARGET(${_target} ${_target})
>+
>+    if (${_target}_OUTPUT_NAME)
>+        set_target_properties(${_target} PROPERTIES OUTPUT_NAME ${${_target}_OUTPUT_NAME})
>+    endif ()
>+endmacro()
>+
>+macro(WEBKIT_WRAP_EXECUTABLE _target)
>+    set(oneValueArgs TARGET_NAME)
>+    set(multiValueArgs SOURCES LIBRARIES)
>+    cmake_parse_arguments(opt "" "${oneValueArgs}" "${multiValueArgs}" ${ARGN})
>+
>+    if (opt_TARGET_NAME)
>+        set(_target_name ${opt_TARGET_NAME})
>+    else ()
>+        set(_target_name ${_target}Lib)
>+    endif ()
>+
>+    add_library(${_target_name} SHARED "${CMAKE_BINARY_DIR}/cmakeconfig.h")
>+
>+    WEBKIT_TARGET(${_target} ${_target_name})
>+
>+    # Unset values
>+    unset(${_target}_HEADERS)
>+    unset(${_target}_INCLUDE_DIRECTORIES)
>+    unset(${_target}_SYSTEM_INCLUDE_DIRECTORIES)
>+    unset(${_target}_PRIVATE_INCLUDE_DIRECTORIES)
>+
>+    # Reset the sources
>+    set(${_target}_SOURCES ${opt_SOURCES})
>+    set(${_target}_LIBRARIES ${opt_LIBRARIES})
>+    set(${_target}_DEPENDENCIES ${_target_name})
>+endmacro()
>+
> macro(WEBKIT_CREATE_FORWARDING_HEADER _target_directory _file)
>     get_filename_component(_source_path "${CMAKE_SOURCE_DIR}/Source/" ABSOLUTE)
>     get_filename_component(_absolute "${_file}" ABSOLUTE)

Actions: View | Formatted Diff | Diff

Attachments on bug 197206: 368050 | 368052 | 368053 | 368054 | 368064 | 368155 | 368175 | 368332 | 368342 | 368347 | 368348 | 368360 | 368467 | 368470