Title: [222110] trunk/Source/WTF
Revision
222110
Author
[email protected]
Date
2017-09-15 14:06:49 -0700 (Fri, 15 Sep 2017)

Log Message

generate-unified-source-bundles.rb shouldn't write a file that isn't going to change
https://bugs.webkit.org/show_bug.cgi?id=177021

Reviewed by Tim Horton.

* generate-unified-source-bundles.rb:

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (222109 => 222110)


--- trunk/Source/WTF/ChangeLog	2017-09-15 20:39:39 UTC (rev 222109)
+++ trunk/Source/WTF/ChangeLog	2017-09-15 21:06:49 UTC (rev 222110)
@@ -1,3 +1,12 @@
+2017-09-15  Keith Miller  <[email protected]>
+
+        generate-unified-source-bundles.rb shouldn't write a file that isn't going to change
+        https://bugs.webkit.org/show_bug.cgi?id=177021
+
+        Reviewed by Tim Horton.
+
+        * generate-unified-source-bundles.rb:
+
 2017-09-14  Saam Barati  <[email protected]>
 
         We should have a way of preventing a caller from making a tail call and we should use it for ProxyObject instead of using build flags

Modified: trunk/Source/WTF/generate-unified-source-bundles.rb (222109 => 222110)


--- trunk/Source/WTF/generate-unified-source-bundles.rb	2017-09-15 20:39:39 UTC (rev 222109)
+++ trunk/Source/WTF/generate-unified-source-bundles.rb	2017-09-15 21:06:49 UTC (rev 222110)
@@ -87,10 +87,13 @@
         @bundleCount += 1
         bundleFile = "UnifiedSource#{@bundleCount}#{extension}"
         bundleFile = $derivedSourcesPath + bundleFile
-        log("writing bundle #{bundleFile} with: \n#{@currentBundleText}")
-        IO::write(bundleFile, @currentBundleText)
         $generatedSources << bundleFile
 
+        if (!bundleFile.exist? || IO::read(bundleFile) != @currentBundleText)
+            log("writing bundle #{bundleFile} with: \n#{@currentBundleText}")
+            IO::write(bundleFile, @currentBundleText)
+        end
+
         @currentBundleText = ""
         @fileCount = 0
     end
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to