Attachment 368278 Details for Bug 197249 – PATCH

[patch] PATCH

197249.diff (text/plain), 5.75 KB, created by Basuke Suzuki on 2019-04-25 15:29:38 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Basuke Suzuki

Created: 2019-04-25 15:29:38 PDT

Size: 5.75 KB

patch

obsolete

>diff --git a/ChangeLog b/ChangeLog
>index a83e5700d2f..896bc8f2ac0 100644
>--- a/ChangeLog
>+++ b/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-04-25  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [Win] Add flag to enable version information stamping and disable by default.
>+        https://bugs.webkit.org/show_bug.cgi?id=197249
>+
>+        Reviewed by Ross Kirsling.
>+
>+        This feature is only used in AppleWin port. Add flag for this task and make it OFF by default.
>+        Then enable it by default on AppleWin.
>+
>+        * Source/cmake/OptionsWin.cmake:
>+
> 2019-04-25  Alex Christensen  <achristensen@webkit.org>
> 
>         Start using C++17
>diff --git a/Source/JavaScriptCore/CMakeLists.txt b/Source/JavaScriptCore/CMakeLists.txt
>index ab69001d005..a12c0748d60 100644
>--- a/Source/JavaScriptCore/CMakeLists.txt
>+++ b/Source/JavaScriptCore/CMakeLists.txt
>@@ -1319,7 +1319,7 @@ endif ()
> add_custom_target(stageSharedScripts DEPENDS ${JavaScriptCore_SCRIPTS})
> add_dependencies(JavaScriptCore stageSharedScripts ${JavaScriptCore_EXTRA_DEPENDENCIES})
> 
>-if (MSVC)
>+if (USE_VERSION_STAMPER)
>     add_custom_command(
>         TARGET JavaScriptCore
>         PRE_BUILD
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index a8ad475f4d4..3ca4bcf01b6 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-04-25  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [Win] Add flag to enable version information stamping and disable by default.
>+        https://bugs.webkit.org/show_bug.cgi?id=197249
>+
>+        Reviewed by Ross Kirsling.
>+
>+        This feature is only used in AppleWin port. Add flag for this task and make it OFF by default.
>+        Then enable it by default on AppleWin.
>+
>+        * CMakeLists.txt:
>+
> 2019-04-25  Timothy Hatcher  <timothy@apple.com>
> 
>         Disable date and time inputs on iOSMac.
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index 19817b6019b..6c62e779d96 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-04-25  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [Win] Add flag to enable version information stamping and disable by default.
>+        https://bugs.webkit.org/show_bug.cgi?id=197249
>+
>+        Reviewed by Ross Kirsling.
>+
>+        This feature is only used in AppleWin port. Add flag for this task and make it OFF by default.
>+        Then enable it by default on AppleWin.
>+
>+        * wtf/CMakeLists.txt:
>+
> 2019-04-25  Timothy Hatcher  <timothy@apple.com>
> 
>         Disable date and time inputs on iOSMac.
>diff --git a/Source/WTF/wtf/CMakeLists.txt b/Source/WTF/wtf/CMakeLists.txt
>index 6ccacc8ca05..5a8bb51b94a 100644
>--- a/Source/WTF/wtf/CMakeLists.txt
>+++ b/Source/WTF/wtf/CMakeLists.txt
>@@ -512,7 +512,9 @@ WEBKIT_MAKE_FORWARDING_HEADERS(WTF
> if (MSVC)
>     set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")
>     set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")
>+endif ()
> 
>+if (USE_VERSION_STAMPER)
>     add_custom_command(
>         TARGET WTF
>         PRE_BUILD
>diff --git a/Source/WebKitLegacy/ChangeLog b/Source/WebKitLegacy/ChangeLog
>index 7e526234ec9..a0cdb3c6813 100644
>--- a/Source/WebKitLegacy/ChangeLog
>+++ b/Source/WebKitLegacy/ChangeLog
>@@ -1,3 +1,15 @@
>+2019-04-25  Basuke Suzuki  <Basuke.Suzuki@sony.com>
>+
>+        [Win] Add flag to enable version information stamping and disable by default.
>+        https://bugs.webkit.org/show_bug.cgi?id=197249
>+
>+        Reviewed by Ross Kirsling.
>+
>+        This feature is only used in AppleWin port. Add flag for this task and make it OFF by default.
>+        Then enable it by default on AppleWin.
>+
>+        * PlatformWin.cmake:
>+
> 2019-04-23  Keith Rollin  <krollin@apple.com>
> 
>         Add Xcode version check for Header post-processing scripts
>diff --git a/Source/WebKitLegacy/PlatformWin.cmake b/Source/WebKitLegacy/PlatformWin.cmake
>index 3506b154db7..d172741f676 100644
>--- a/Source/WebKitLegacy/PlatformWin.cmake
>+++ b/Source/WebKitLegacy/PlatformWin.cmake
>@@ -395,11 +395,13 @@ set(WEBKITLEGACY_IDL_DEPENDENCIES
>     "${WebKitLegacy_DERIVED_SOURCES_DIR}/include/autoversion.h"
> )
> 
>-add_custom_command(
>-    OUTPUT ${WebKitLegacy_DERIVED_SOURCES_DIR}/include/autoversion.h
>-    WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
>-    COMMAND ${PERL_EXECUTABLE} ${WEBKIT_LIBRARIES_DIR}/tools/scripts/auto-version.pl ${WebKitLegacy_DERIVED_SOURCES_DIR}
>-    VERBATIM)
>+if (USE_VERSION_STAMPER)
>+    add_custom_command(
>+        OUTPUT ${WebKitLegacy_DERIVED_SOURCES_DIR}/include/autoversion.h
>+        WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
>+        COMMAND ${PERL_EXECUTABLE} ${WEBKIT_LIBRARIES_DIR}/tools/scripts/auto-version.pl ${WebKitLegacy_DERIVED_SOURCES_DIR}
>+        VERBATIM)
>+endif ()
> 
> GENERATE_INTERFACE(win/Interfaces/WebKit.idl ${MIDL_DEFINES} "${WEBKITLEGACY_IDL_DEPENDENCIES}")
> GENERATE_INTERFACE(win/Interfaces/Accessible2/AccessibleApplication.idl ${MIDL_DEFINES} "${WEBKITLEGACY_IDL_DEPENDENCIES}")
>diff --git a/Source/cmake/OptionsWin.cmake b/Source/cmake/OptionsWin.cmake
>index 0b46991befb..d89cbff7e7e 100644
>--- a/Source/cmake/OptionsWin.cmake
>+++ b/Source/cmake/OptionsWin.cmake
>@@ -14,6 +14,8 @@ endif ()
> 
> WEBKIT_OPTION_BEGIN()
> 
>+WEBKIT_OPTION_DEFINE(USE_VERSION_STAMPER "Toggle stamping version information during build" PRIVATE OFF)
>+
> # FIXME: Most of these options should not be public.
> WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_3D_TRANSFORMS PUBLIC ON)
> WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_ACCELERATED_2D_CANVAS PUBLIC OFF)
>@@ -98,6 +100,7 @@ else ()
>     WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_INTL PUBLIC OFF)
>     WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_PUBLIC_SUFFIX_LIST PRIVATE OFF)
>     WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_WEBGL PUBLIC OFF)
>+    WEBKIT_OPTION_DEFAULT_PORT_VALUE(USE_VERSION_STAMPER PRIVATE ON)
> endif ()
> 
> WEBKIT_OPTION_END()

Actions: View | Formatted Diff | Diff

Attachments on bug 197249: 368186 | 368256 | 368278 | 368363 | 368364 | 368466