Title: [203111] trunk
Revision
203111
Author
[email protected]
Date
2016-07-12 04:14:43 -0700 (Tue, 12 Jul 2016)

Log Message

[Win] DLLs are missing version information.
https://bugs.webkit.org/show_bug.cgi?id=159349

Reviewed by Brent Fulgham.

.:

The version stamping will fail if the target path has forward slashes.

* Source/cmake/tools/scripts/version-stamp.pl: Replace forward slashes with backslashes.

Source/_javascript_Core:

Generate autoversion.h and run perl version stamp utility.

* CMakeLists.txt:

Source/WebKit:

Run perl version stamp utility.

* CMakeLists.txt:

Source/WTF:

Generate autoversion.h and run perl version stamp utility.

* wtf/CMakeLists.txt:

WebKitLibraries:

The version stamping will fail if the target path has forward slashes.

* win/tools/scripts/version-stamp.pl: Replace forward slashes with backslashes.

Modified Paths

Diff

Modified: trunk/ChangeLog (203110 => 203111)


--- trunk/ChangeLog	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/ChangeLog	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1,3 +1,14 @@
+2016-07-12  Per Arne Vollan  <[email protected]>
+
+        [Win] DLLs are missing version information.
+        https://bugs.webkit.org/show_bug.cgi?id=159349
+
+        Reviewed by Brent Fulgham.
+
+        The version stamping will fail if the target path has forward slashes.
+
+        * Source/cmake/tools/scripts/version-stamp.pl: Replace forward slashes with backslashes.
+
 2016-07-05  Olivier Blin  <[email protected]>
 
         ENABLE_MEDIA_SOURCE should depend on ENABLE_VIDEO

Modified: trunk/Source/_javascript_Core/CMakeLists.txt (203110 => 203111)


--- trunk/Source/_javascript_Core/CMakeLists.txt	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/_javascript_Core/CMakeLists.txt	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1354,3 +1354,17 @@
 
 add_custom_target(stageSharedScripts DEPENDS ${_javascript_Core_SCRIPTS})
 add_dependencies(_javascript_Core stageSharedScripts)
+
+if (MSVC)
+    add_custom_command(
+        TARGET _javascript_Core
+        PRE_BUILD
+        COMMAND ${PERL_EXECUTABLE} ${WEBKIT_LIBRARIES_DIR}/tools/scripts/auto-version.pl ${DERIVED_SOURCES_DIR}
+        VERBATIM)
+
+    add_custom_command(
+        TARGET _javascript_Core
+        POST_BUILD
+        COMMAND ${PERL_EXECUTABLE} ${CMAKE_SOURCE_DIR}/WebKitLibraries/win/tools/scripts/version-stamp.pl ${DERIVED_SOURCES_DIR} $<TARGET_FILE:_javascript_Core>
+        VERBATIM)
+endif ()

Modified: trunk/Source/_javascript_Core/ChangeLog (203110 => 203111)


--- trunk/Source/_javascript_Core/ChangeLog	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1,3 +1,14 @@
+2016-07-12  Per Arne Vollan  <[email protected]>
+
+        [Win] DLLs are missing version information.
+        https://bugs.webkit.org/show_bug.cgi?id=159349
+
+        Reviewed by Brent Fulgham.
+
+        Generate autoversion.h and run perl version stamp utility.
+
+        * CMakeLists.txt:
+
 2016-07-11  Caio Lima  <[email protected]>
 
         ECMAScript 2016: %TypedArray%.prototype.includes implementation

Modified: trunk/Source/WTF/ChangeLog (203110 => 203111)


--- trunk/Source/WTF/ChangeLog	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/WTF/ChangeLog	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1,3 +1,14 @@
+2016-07-12  Per Arne Vollan  <[email protected]>
+
+        [Win] DLLs are missing version information.
+        https://bugs.webkit.org/show_bug.cgi?id=159349
+
+        Reviewed by Brent Fulgham.
+
+        Generate autoversion.h and run perl version stamp utility.
+
+        * wtf/CMakeLists.txt:
+
 2016-07-11  Myles C. Maxfield  <[email protected]>
 
         Implement grapheme cluster iterator on StringView

Modified: trunk/Source/WTF/wtf/CMakeLists.txt (203110 => 203111)


--- trunk/Source/WTF/wtf/CMakeLists.txt	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/WTF/wtf/CMakeLists.txt	2016-07-12 11:14:43 UTC (rev 203111)
@@ -307,4 +307,16 @@
 if (MSVC)
     set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")
     set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} /SUBSYSTEM:WINDOWS")
+
+    add_custom_command(
+        TARGET WTF
+        PRE_BUILD
+        COMMAND ${PERL_EXECUTABLE} ${WEBKIT_LIBRARIES_DIR}/tools/scripts/auto-version.pl ${DERIVED_SOURCES_DIR}
+        VERBATIM)
+
+    add_custom_command(
+        TARGET WTF
+        POST_BUILD
+        COMMAND ${PERL_EXECUTABLE} ${CMAKE_SOURCE_DIR}/WebKitLibraries/win/tools/scripts/version-stamp.pl ${DERIVED_SOURCES_DIR} $<TARGET_FILE:WTF>
+        VERBATIM)
 endif ()

Modified: trunk/Source/WebKit/CMakeLists.txt (203110 => 203111)


--- trunk/Source/WebKit/CMakeLists.txt	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/WebKit/CMakeLists.txt	2016-07-12 11:14:43 UTC (rev 203111)
@@ -56,3 +56,11 @@
 if (APPLE)
     set_target_properties(WebKit PROPERTIES LINK_FLAGS "-umbrella WebKit")
 endif ()
+
+if (MSVC)
+    add_custom_command(
+        TARGET WebKit
+        POST_BUILD
+        COMMAND ${PERL_EXECUTABLE} ${CMAKE_SOURCE_DIR}/WebKitLibraries/win/tools/scripts/version-stamp.pl ${DERIVED_SOURCES_WEBKIT_DIR} $<TARGET_FILE:WebKit>
+        VERBATIM)
+endif ()

Modified: trunk/Source/WebKit/ChangeLog (203110 => 203111)


--- trunk/Source/WebKit/ChangeLog	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/WebKit/ChangeLog	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1,3 +1,14 @@
+2016-07-12  Per Arne Vollan  <[email protected]>
+
+        [Win] DLLs are missing version information.
+        https://bugs.webkit.org/show_bug.cgi?id=159349
+
+        Reviewed by Brent Fulgham.
+
+        Run perl version stamp utility.
+
+        * CMakeLists.txt:
+
 2016-07-11  Commit Queue  <[email protected]>
 
         Unreviewed, rolling out r203064.

Modified: trunk/Source/cmake/tools/scripts/version-stamp.pl (203110 => 203111)


--- trunk/Source/cmake/tools/scripts/version-stamp.pl	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/Source/cmake/tools/scripts/version-stamp.pl	2016-07-12 11:14:43 UTC (rev 203111)
@@ -89,6 +89,9 @@
 
 my $TARGET_PATH = File::Spec->canonpath($target);
 
+# Replace forward slashes with backslashes in case we are running Cygwin Perl.
+$TARGET_PATH =~ s/\//\\/g;
+
 print "Adjusting RC_PROJECTSOURCEVERSION and RC_ProjectSourceVersion to be safe for VersionStamper.\n";
 
 my $SAFE_PROJECT_VERSION = "$components{'__FULL_BUILD_MAJOR_VERSION__'}.$components{'__VERSION_MINOR__'}.$components{'__VERSION_TINY__'}";

Modified: trunk/WebKitLibraries/ChangeLog (203110 => 203111)


--- trunk/WebKitLibraries/ChangeLog	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/WebKitLibraries/ChangeLog	2016-07-12 11:14:43 UTC (rev 203111)
@@ -1,3 +1,14 @@
+2016-07-12  Per Arne Vollan  <[email protected]>
+
+        [Win] DLLs are missing version information.
+        https://bugs.webkit.org/show_bug.cgi?id=159349
+
+        Reviewed by Brent Fulgham.
+
+        The version stamping will fail if the target path has forward slashes.
+
+        * win/tools/scripts/version-stamp.pl: Replace forward slashes with backslashes.
+
 2016-06-14  Lucas Forschler  <[email protected]>
 
         Update existing WebKitSystemInterface Libraries.

Modified: trunk/WebKitLibraries/win/tools/scripts/version-stamp.pl (203110 => 203111)


--- trunk/WebKitLibraries/win/tools/scripts/version-stamp.pl	2016-07-12 09:04:49 UTC (rev 203110)
+++ trunk/WebKitLibraries/win/tools/scripts/version-stamp.pl	2016-07-12 11:14:43 UTC (rev 203111)
@@ -89,6 +89,9 @@
 
 my $TARGET_PATH = File::Spec->canonpath($target);
 
+# Replace forward slashes with backslashes in case we are running Cygwin Perl.
+$TARGET_PATH =~ s/\//\\/g;
+
 print "Adjusting RC_PROJECTSOURCEVERSION and RC_ProjectSourceVersion to be safe for VersionStamper.\n";
 
 my $SAFE_PROJECT_VERSION = "$components{'__FULL_BUILD_MAJOR_VERSION__'}.$components{'__VERSION_MINOR__'}.$components{'__VERSION_TINY__'}";
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to