Title: [203181] tags/Safari-602.1.41

Diff

Modified: tags/Safari-602.1.41/ChangeLog (203180 => 203181)


--- tags/Safari-602.1.41/ChangeLog	2016-07-13 20:25:23 UTC (rev 203180)
+++ tags/Safari-602.1.41/ChangeLog	2016-07-13 20:26:58 UTC (rev 203181)
@@ -1,3 +1,20 @@
+2016-07-13  Babak Shafiei  <[email protected]>
+
+        Merge r203167.
+
+    2016-07-13  Per Arne Vollan  <[email protected]>
+
+            [Win] DLLs are missing version information.
+            https://bugs.webkit.org/show_bug.cgi?id=159349
+
+            Reviewed by Alex Christensen.
+
+            Avoid using environment variable WEBKIT_LIBRARIES when finding version stamper utility,
+            in case it is not defined. Instead, use the location of the perl script to find the
+            utility.
+
+            * Source/cmake/tools/scripts/version-stamp.pl:
+
 2016-07-12  Gyuyoung Kim  <[email protected]>
 
         Remove ENABLE_CSS3_TEXT_LINE_BREAK flag

Modified: tags/Safari-602.1.41/Source/cmake/tools/scripts/version-stamp.pl (203180 => 203181)


--- tags/Safari-602.1.41/Source/cmake/tools/scripts/version-stamp.pl	2016-07-13 20:25:23 UTC (rev 203180)
+++ tags/Safari-602.1.41/Source/cmake/tools/scripts/version-stamp.pl	2016-07-13 20:26:58 UTC (rev 203181)
@@ -1,6 +1,8 @@
 #!/usr/bin/perl -w
 
 use strict;
+use Cwd 'abs_path';
+use File::Basename;
 use File::Spec;
 use POSIX;
 
@@ -29,10 +31,12 @@
 
 sub exitStatus($);
 
-my $WEBKIT_LIBRARIES = $ENV{'WEBKIT_LIBRARIES'};
+my $rawPath = $0;
 
-my $VERSION_STAMPER = File::Spec->catfile($WEBKIT_LIBRARIES, 'tools', 'VersionStamper', 'VersionStamper.exe');
+my $thisDirectory = dirname(abs_path($rawPath));
 
+my $VERSION_STAMPER = File::Spec->catfile($thisDirectory, '..', 'VersionStamper', 'VersionStamper.exe');
+
 unless (-e $VERSION_STAMPER) {
     print "No $VERSION_STAMPER executable. Exiting.\n";
     exit(0);

Modified: tags/Safari-602.1.41/WebKitLibraries/ChangeLog (203180 => 203181)


--- tags/Safari-602.1.41/WebKitLibraries/ChangeLog	2016-07-13 20:25:23 UTC (rev 203180)
+++ tags/Safari-602.1.41/WebKitLibraries/ChangeLog	2016-07-13 20:26:58 UTC (rev 203181)
@@ -1,3 +1,20 @@
+2016-07-13  Babak Shafiei  <[email protected]>
+
+        Merge r203167.
+
+    2016-07-13  Per Arne Vollan  <[email protected]>
+
+            [Win] DLLs are missing version information.
+            https://bugs.webkit.org/show_bug.cgi?id=159349
+
+            Reviewed by Alex Christensen.
+
+            Avoid using environment variable WEBKIT_LIBRARIES when finding version stamper utility,
+            in case it is not defined. Instead, use the location of the perl script to find the
+            utility.
+
+            * win/tools/scripts/version-stamp.pl:
+
 2016-07-12  Per Arne Vollan  <[email protected]>
 
         [Win] DLLs are missing version information.

Modified: tags/Safari-602.1.41/WebKitLibraries/win/tools/scripts/version-stamp.pl (203180 => 203181)


--- tags/Safari-602.1.41/WebKitLibraries/win/tools/scripts/version-stamp.pl	2016-07-13 20:25:23 UTC (rev 203180)
+++ tags/Safari-602.1.41/WebKitLibraries/win/tools/scripts/version-stamp.pl	2016-07-13 20:26:58 UTC (rev 203181)
@@ -1,6 +1,8 @@
 #!/usr/bin/perl -w
 
 use strict;
+use Cwd 'abs_path';
+use File::Basename;
 use File::Spec;
 use POSIX;
 
@@ -29,10 +31,12 @@
 
 sub exitStatus($);
 
-my $WEBKIT_LIBRARIES = $ENV{'WEBKIT_LIBRARIES'};
+my $rawPath = $0;
 
-my $VERSION_STAMPER = File::Spec->catfile($WEBKIT_LIBRARIES, 'tools', 'VersionStamper', 'VersionStamper.exe');
+my $thisDirectory = dirname(abs_path($rawPath));
 
+my $VERSION_STAMPER = File::Spec->catfile($thisDirectory, '..', 'VersionStamper', 'VersionStamper.exe');
+
 unless (-e $VERSION_STAMPER) {
     print "No $VERSION_STAMPER executable. Exiting.\n";
     exit(0);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to