[Win] Add flag to enable version information stamping and disable by default.
authorBasuke.Suzuki@sony.com <Basuke.Suzuki@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Apr 2019 18:18:54 +0000 (18:18 +0000)
committerBasuke.Suzuki@sony.com <Basuke.Suzuki@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 29 Apr 2019 18:18:54 +0000 (18:18 +0000)
https://bugs.webkit.org/show_bug.cgi?id=197249
<rdar://problem/50224412>

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:

Source/JavaScriptCore:

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:

Source/WTF:

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:

WebKitLibraries:

Modify script not to invoke SVN command if the directory for SVN doesn't exists.

* win/tools/scripts/auto-version.pl:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@244741 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
Source/JavaScriptCore/CMakeLists.txt
Source/JavaScriptCore/ChangeLog
Source/WTF/ChangeLog
Source/WTF/wtf/CMakeLists.txt
Source/cmake/OptionsWin.cmake
WebKitLibraries/ChangeLog
WebKitLibraries/win/tools/scripts/auto-version.pl

index b4a5d16..319e3da 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2019-04-29  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
+        <rdar://problem/50224412>
+
+        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-28  Andy Estes  <aestes@apple.com>
 
         Fix the watchOS engineering build.
index ab69001..a12c074 100644 (file)
@@ -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
index 28aeca4..d2ced41 100644 (file)
@@ -1,3 +1,16 @@
+2019-04-29  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
+        <rdar://problem/50224412>
+
+        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-26  Keith Rollin  <krollin@apple.com>
 
         Enable new build rule for post-processing headers when using XCBuild
index 475cdd2..3fe139c 100644 (file)
@@ -1,3 +1,16 @@
+2019-04-29  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
+        <rdar://problem/50224412>
+
+        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-26  Don Olmstead  <don.olmstead@sony.com>
 
         Add WTF::findIgnoringASCIICaseWithoutLength to replace strcasestr
index 6ccacc8..5a8bb51 100644 (file)
@@ -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
index 0b46991..d89cbff 100644 (file)
@@ -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()
index b011b47..4a46faf 100644 (file)
@@ -1,3 +1,15 @@
+2019-04-29  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
+        <rdar://problem/50224412>
+
+        Reviewed by Ross Kirsling.
+
+        Modify script not to invoke SVN command if the directory for SVN doesn't exists.
+
+        * win/tools/scripts/auto-version.pl:
+
 2019-04-12  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         [iOS] Software keyboard is shown too frequently on some websites
index a70cc86..f6827fe 100755 (executable)
@@ -83,9 +83,12 @@ if (!$ENVIRONMENT_VERSION) {
     # an engineering build and we'll stamp the build with some more information.
 
     my $BUILD_DATE = localtime(time);
-    my $SVN_REVISION = `svn info`;
-    $SVN_REVISION =~ m/Revision: (\d+)/;
-    $SVN_REVISION= $1;
+    my $SVN_REVISION = '';
+    if (-d File::Spec->catfile($thisDirectory, '.svn')) {
+        $SVN_REVISION = `svn info`;
+        $SVN_REVISION =~ m/Revision: (\d+)/;
+        $SVN_REVISION= $1;
+    }
 
     chomp($BUILD_DATE);
     chomp($SVN_REVISION);