Title: [99376] trunk/Tools
Revision
99376
Author
[email protected]
Date
2011-11-06 09:22:14 -0800 (Sun, 06 Nov 2011)

Log Message

Made show-pretty-diff work when the input path contains spaces.

Reviewed by Anders Carlsson.

* Scripts/show-pretty-diff:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (99375 => 99376)


--- trunk/Tools/ChangeLog	2011-11-06 11:54:59 UTC (rev 99375)
+++ trunk/Tools/ChangeLog	2011-11-06 17:22:14 UTC (rev 99376)
@@ -1,3 +1,11 @@
+2011-11-06  Dan Bernstein  <[email protected]>
+
+        Made show-pretty-diff work when the input path contains spaces.
+
+        Reviewed by Anders Carlsson.
+
+        * Scripts/show-pretty-diff:
+
 2011-11-05  Daniel Cheng  <[email protected]>
 
         [chromium] Use the security origin instead of the URL when checking notification permissions

Modified: trunk/Tools/Scripts/show-pretty-diff (99375 => 99376)


--- trunk/Tools/Scripts/show-pretty-diff	2011-11-06 11:54:59 UTC (rev 99375)
+++ trunk/Tools/Scripts/show-pretty-diff	2011-11-06 17:22:14 UTC (rev 99376)
@@ -64,7 +64,7 @@
 my $prettyPatchTool = sourceDir() . "/Websites/bugs.webkit.org/PrettyPatch/prettify.rb";
 
 my $pathToPrettify = "ruby -I " . sourceDir() . "/Websites/bugs.webkit.org/PrettyPatch/ " . sourceDir() . "/Websites/bugs.webkit.org/PrettyPatch/prettify.rb";
-system "$pathToPrettify $inputPath > $prettydiffPath";
+system "$pathToPrettify " . quotemeta($inputPath) . " > $prettydiffPath";
 
 if (isAppleMacWebKit()) {
     system "open", $prettydiffPath;
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to