Changeset 257565 in webkit


Ignore:
Timestamp:
Feb 27, 2020 6:36:58 AM (4 years ago)
Author:
don.olmstead@sony.com
Message:

[CMake] Use WEBKIT_EXECUTABLE in MiniBrowser
https://bugs.webkit.org/show_bug.cgi?id=206894

Reviewed by Adrian Perez de Castro.

.:

Add a MiniBrowser derived sources directory.

  • Source/cmake/WebKitFS.cmake:

Tools:

Modify CMake ports to use WEBKIT_EXECUTABLE when creating their respective
MiniBrowsers.

  • MiniBrowser/gtk/CMakeLists.txt:
  • MiniBrowser/win/CMakeLists.txt:
  • MiniBrowser/wpe/CMakeLists.txt:
Location:
trunk
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/ChangeLog

    r257562 r257565  
     12020-02-27  Don Olmstead  <don.olmstead@sony.com>
     2
     3        [CMake] Use WEBKIT_EXECUTABLE in MiniBrowser
     4        https://bugs.webkit.org/show_bug.cgi?id=206894
     5
     6        Reviewed by Adrian Perez de Castro.
     7
     8        Add a MiniBrowser derived sources directory.
     9
     10        * Source/cmake/WebKitFS.cmake:
     11
    1122020-02-27  Adrian Perez de Castro  <aperez@igalia.com>
    213
  • trunk/Source/cmake/WebKitFS.cmake

    r245723 r257565  
    4848set(WebKit_DERIVED_SOURCES_DIR "${CMAKE_BINARY_DIR}/DerivedSources/WebKit")
    4949set(WebInspectorUI_DERIVED_SOURCES_DIR "${CMAKE_BINARY_DIR}/DerivedSources/WebInspectorUI")
     50set(MiniBrowser_DERIVED_SOURCES_DIR "${CMAKE_BINARY_DIR}/DerivedSources/MiniBrowser")
    5051
    5152set(FORWARDING_HEADERS_DIR ${DERIVED_SOURCES_DIR}/ForwardingHeaders)
  • trunk/Tools/ChangeLog

    r257553 r257565  
     12020-02-27  Don Olmstead  <don.olmstead@sony.com>
     2
     3        [CMake] Use WEBKIT_EXECUTABLE in MiniBrowser
     4        https://bugs.webkit.org/show_bug.cgi?id=206894
     5
     6        Reviewed by Adrian Perez de Castro.
     7
     8        Modify CMake ports to use WEBKIT_EXECUTABLE when creating their respective
     9        MiniBrowsers.
     10
     11        * MiniBrowser/gtk/CMakeLists.txt:
     12        * MiniBrowser/win/CMakeLists.txt:
     13        * MiniBrowser/wpe/CMakeLists.txt:
     14
    1152020-02-26  Alex Christensen  <achristensen@webkit.org>
    216
  • trunk/Tools/MiniBrowser/gtk/CMakeLists.txt

    r228818 r257565  
    1 set(MINIBROWSER_DIR "${TOOLS_DIR}/MiniBrowser/gtk")
    2 set(DERIVED_SOURCES_MINIBROWSER_DIR "${CMAKE_BINARY_DIR}/DerivedSources/MiniBrowser")
     1set(MiniBrowser_DIR "${TOOLS_DIR}/MiniBrowser/gtk")
    32
    4 file(MAKE_DIRECTORY ${DERIVED_SOURCES_MINIBROWSER_DIR})
     3file(MAKE_DIRECTORY ${MiniBrowser_DERIVED_SOURCES_DIR})
    54
    65set(MiniBrowser_SOURCES
    7     ${DERIVED_SOURCES_MINIBROWSER_DIR}/BrowserMarshal.c
    8     ${MINIBROWSER_DIR}/BrowserCellRendererVariant.c
    9     ${MINIBROWSER_DIR}/BrowserCellRendererVariant.h
    10     ${MINIBROWSER_DIR}/BrowserDownloadsBar.c
    11     ${MINIBROWSER_DIR}/BrowserDownloadsBar.h
    12     ${MINIBROWSER_DIR}/BrowserSearchBar.c
    13     ${MINIBROWSER_DIR}/BrowserSearchBar.h
    14     ${MINIBROWSER_DIR}/BrowserSettingsDialog.c
    15     ${MINIBROWSER_DIR}/BrowserSettingsDialog.h
    16     ${MINIBROWSER_DIR}/BrowserTab.c
    17     ${MINIBROWSER_DIR}/BrowserTab.h
    18     ${MINIBROWSER_DIR}/BrowserWindow.c
    19     ${MINIBROWSER_DIR}/BrowserWindow.h
    20     ${MINIBROWSER_DIR}/main.c
     6    ${MiniBrowser_DERIVED_SOURCES_DIR}/BrowserMarshal.c
     7    BrowserCellRendererVariant.c
     8    BrowserCellRendererVariant.h
     9    BrowserDownloadsBar.c
     10    BrowserDownloadsBar.h
     11    BrowserSearchBar.c
     12    BrowserSearchBar.h
     13    BrowserSettingsDialog.c
     14    BrowserSettingsDialog.h
     15    BrowserTab.c
     16    BrowserTab.h
     17    BrowserWindow.c
     18    BrowserWindow.h
     19    main.c
    2120)
    2221
    23 set(MiniBrowser_INCLUDE_DIRECTORIES
    24     ${DERIVED_SOURCES_MINIBROWSER_DIR}
     22set(MiniBrowser_PRIVATE_INCLUDE_DIRECTORIES
     23    ${CMAKE_BINRARY_DIR}
     24    ${MiniBrowser_DERIVED_SOURCES_DIR}
    2525    ${DERIVED_SOURCES_WEBKIT2GTK_DIR}
    2626    ${FORWARDING_HEADERS_WEBKIT2GTK_DIR}
     
    3636)
    3737
    38 set(MiniBrowser_LIBRARIES
     38set(MiniBrowser_PRIVATE_LIBRARIES
    3939    ${JavaScriptCore_LIBRARY_NAME}
    4040    WebKit
     
    4545)
    4646
     47set(MiniBrowser_DEPENDENCIES
     48    NetworkProcess
     49    WebProcess
     50)
     51
    4752add_custom_command(
    48     OUTPUT ${DERIVED_SOURCES_MINIBROWSER_DIR}/BrowserMarshal.c
    49            ${DERIVED_SOURCES_MINIBROWSER_DIR}/BrowserMarshal.h
    50     MAIN_DEPENDENCY ${MINIBROWSER_DIR}/browser-marshal.list
    51     COMMAND glib-genmarshal --prefix=browser_marshal ${MINIBROWSER_DIR}/browser-marshal.list --body > ${DERIVED_SOURCES_MINIBROWSER_DIR}/BrowserMarshal.c
    52     COMMAND glib-genmarshal --prefix=browser_marshal ${MINIBROWSER_DIR}/browser-marshal.list --header > ${DERIVED_SOURCES_MINIBROWSER_DIR}/BrowserMarshal.h
     53    OUTPUT ${MiniBrowser_DERIVED_SOURCES_DIR}/BrowserMarshal.c
     54           ${MiniBrowser_DERIVED_SOURCES_DIR}/BrowserMarshal.h
     55    MAIN_DEPENDENCY ${MiniBrowser_DIR}/browser-marshal.list
     56    COMMAND glib-genmarshal --prefix=browser_marshal ${MiniBrowser_DIR}/browser-marshal.list --body > ${MiniBrowser_DERIVED_SOURCES_DIR}/BrowserMarshal.c
     57    COMMAND glib-genmarshal --prefix=browser_marshal ${MiniBrowser_DIR}/browser-marshal.list --header > ${MiniBrowser_DERIVED_SOURCES_DIR}/BrowserMarshal.h
    5358    VERBATIM)
    5459
     60set(MiniBrowser_PRIVATE_DEFINITIONS GDK_VERSION_MIN_REQUIRED=GDK_VERSION_3_6)
    5561if (DEVELOPER_MODE)
    56     add_definitions(-DWEBKIT_INJECTED_BUNDLE_PATH="${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
     62    list(APPEND MiniBrowser_PRIVATE_DEFINITIONS -DWEBKIT_INJECTED_BUNDLE_PATH="${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
    5763endif ()
    5864
    59 add_definitions(-DGDK_VERSION_MIN_REQUIRED=GDK_VERSION_3_6)
    60 
    61 include_directories(${MiniBrowser_INCLUDE_DIRECTORIES})
    62 include_directories(SYSTEM ${MiniBrowser_SYSTEM_INCLUDE_DIRECTORIES})
    63 add_executable(MiniBrowser ${MiniBrowser_SOURCES})
    64 target_link_libraries(MiniBrowser ${MiniBrowser_LIBRARIES})
    65 
     65WEBKIT_EXECUTABLE_DECLARE(MiniBrowser)
     66WEBKIT_EXECUTABLE(MiniBrowser)
    6667WEBKIT_ADD_TARGET_CXX_FLAGS(MiniBrowser -Wno-unused-parameter)
    6768
  • trunk/Tools/MiniBrowser/win/CMakeLists.txt

    r252872 r257565  
    1 set(MiniBrowser_INCLUDE_DIRECTORIES
    2     ${PAL_FRAMEWORK_HEADERS_DIR}
    3     ${WebCore_PRIVATE_FRAMEWORK_HEADERS_DIR}
     1set(MiniBrowser_PRIVATE_INCLUDE_DIRECTORIES
     2    ${CMAKE_BINARY_DIR}
    43)
    54
     
    1312)
    1413
    15 set(MiniBrowser_LIBRARIES
    16     DbgHelp
     14set(MiniBrowser_PRIVATE_DEFINITIONS _UNICODE)
     15set(MiniBrowser_PRIVATE_LIBRARIES
    1716    WebKit::WTF
    1817    comctl32
    19     comsupp
    20     comsuppw
    2118    shlwapi
     19    user32
    2220)
    2321
     22if (${WTF_PLATFORM_WIN_CAIRO})
     23    list(APPEND MiniBrowser_PRIVATE_DEFINITIONS WIN_CAIRO)
     24endif ()
     25
    2426if (USE_CF)
    25     list(APPEND MiniBrowser_LIBRARIES
     27    list(APPEND MiniBrowser_PRIVATE_LIBRARIES
    2628        ${COREFOUNDATION_LIBRARY}
    2729    )
     
    2931
    3032if (ENABLE_WEBKIT_LEGACY)
    31     add_definitions(-DENABLE_WEBKIT_LEGACY)
     33    list(APPEND MiniBrowser_PRIVATE_DEFINITIONS ENABLE_WEBKIT_LEGACY)
    3234    list(APPEND MiniBrowser_INCLUDE_DIRECTORIES
    3335        ${WebKitLegacy_FRAMEWORK_HEADERS_DIR}
     
    4143        WebKitLegacyBrowserWindow.cpp
    4244    )
    43     list(APPEND MiniBrowser_LIBRARIES
     45    list(APPEND MiniBrowser_PRIVATE_LIBRARIES
    4446        WebKitLegacy
    4547        WebKitLegacyGUID
     
    4850
    4951if (ENABLE_WEBKIT)
    50     add_definitions(-DENABLE_WEBKIT)
     52    list(APPEND MiniBrowser_PRIVATE_DEFINITIONS ENABLE_WEBKIT)
    5153    list(APPEND MiniBrowser_INCLUDE_DIRECTORIES
    5254        ${WebKit_FRAMEWORK_HEADERS_DIR}
     
    5658        WebKitBrowserWindow.cpp
    5759    )
    58     list(APPEND MiniBrowser_LIBRARIES
     60    list(APPEND MiniBrowser_PRIVATE_LIBRARIES
    5961        WebKit
     62    )
     63    list(APPEND MiniBrowser_DEPENDENCIES
     64        NetworkProcess
     65        WebKitFrameworkHeaders
     66        WebProcess
    6067    )
    6168endif ()
    6269
    63 set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} ${MSVC_RUNTIME_LINKER_FLAGS}")
    64 set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /ENTRY:wWinMainCRTStartup")
    65 
    66 if (${WTF_PLATFORM_WIN_CAIRO})
    67     add_definitions(-DWIN_CAIRO)
    68 endif ()
    69 add_definitions(-D_UNICODE)
    70 include_directories(${MiniBrowser_INCLUDE_DIRECTORIES})
    71 add_library(MiniBrowserLib SHARED ${MiniBrowser_SOURCES})
    72 target_link_libraries(MiniBrowserLib ${MiniBrowser_LIBRARIES})
    73 
    74 add_executable(MiniBrowser WIN32 ${TOOLS_DIR}/win/DLLLauncher/DLLLauncherMain.cpp MiniBrowser.rc)
    75 target_link_libraries(MiniBrowser shlwapi)
    76 set_target_properties(MiniBrowser PROPERTIES OUTPUT_NAME "MiniBrowser")
    77 
    78 add_dependencies(MiniBrowser MiniBrowserLib)
     70WEBKIT_EXECUTABLE_DECLARE(MiniBrowser)
     71WEBKIT_WRAP_EXECUTABLE(MiniBrowser
     72    SOURCES ${TOOLS_DIR}/win/DLLLauncher/DLLLauncherMain.cpp MiniBrowser.rc
     73    LIBRARIES shlwapi
     74)
     75WEBKIT_EXECUTABLE(MiniBrowser)
     76set_target_properties(MiniBrowser PROPERTIES WIN32_EXECUTABLE ON)
  • trunk/Tools/MiniBrowser/wpe/CMakeLists.txt

    r257053 r257565  
    1 set(MINIBROWSER_DIR "${TOOLS_DIR}/MiniBrowser/wpe")
    2 
    31set(MiniBrowser_SOURCES
    4     ${MINIBROWSER_DIR}/main.cpp
     2    main.cpp
    53)
    64
    7 set(MiniBrowser_INCLUDE_DIRECTORIES
     5set(MiniBrowser_PRIVATE_INCLUDE_DIRECTORIES
     6    ${CMAKE_BINARY_DIR}
    87    ${TOOLS_DIR}/wpe/backends/
    98    ${DERIVED_SOURCES_WPE_API_DIR}
     
    1716)
    1817
    19 set(MiniBrowser_LIBRARIES
     18set(MiniBrowser_PRIVATE_LIBRARIES
    2019    WPE::libwpe
    2120    WPEToolingBackends
     
    2625)
    2726
     27set(MiniBrowser_DEPENDENCIES
     28    NetworkProcess
     29    WebProcess
     30)
     31
    2832if (DEVELOPER_MODE)
    29     add_definitions(-DWEBKIT_INJECTED_BUNDLE_PATH="${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
     33    list(APPEND MiniBrowser_PRIVATE_DEFINITIONS WEBKIT_INJECTED_BUNDLE_PATH="${CMAKE_LIBRARY_OUTPUT_DIRECTORY}")
    3034endif ()
    3135
    32 include_directories(${MiniBrowser_INCLUDE_DIRECTORIES})
    33 include_directories(SYSTEM ${MiniBrowser_SYSTEM_INCLUDE_DIRECTORIES})
    34 add_executable(MiniBrowser ${MiniBrowser_SOURCES})
    35 target_link_libraries(MiniBrowser ${MiniBrowser_LIBRARIES})
     36WEBKIT_EXECUTABLE_DECLARE(MiniBrowser)
     37WEBKIT_EXECUTABLE(MiniBrowser)
    3638
    3739install(TARGETS MiniBrowser DESTINATION "${LIBEXEC_INSTALL_DIR}")
Note: See TracChangeset for help on using the changeset viewer.