2011-01-12 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jan 2011 23:48:54 +0000 (23:48 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jan 2011 23:48:54 +0000 (23:48 +0000)
        Reviewed by Adam Barth.

        review tool formatted diff doesn't match the uploaded diff
        https://bugs.webkit.org/show_bug.cgi?id=51960

        PrettyPatch works just fine if there is no newline at the end of the file.

        * attachment.cgi:

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

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/attachment.cgi

index 6b5f678..91789dd 100644 (file)
@@ -2,6 +2,17 @@
 
         Reviewed by Adam Barth.
 
+        review tool formatted diff doesn't match the uploaded diff
+        https://bugs.webkit.org/show_bug.cgi?id=51960
+
+        PrettyPatch works just fine if there is no newline at the end of the file.
+
+        * attachment.cgi:
+
+2011-01-12  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
         Bugzilla: Add keyboard shortcuts to jump to next change
         https://bugs.webkit.org/show_bug.cgi?id=52305
 
index 90c279d..2427b59 100755 (executable)
@@ -396,7 +396,7 @@ sub prettyPatch
     $ENV{'PATH'} = "/opt/local/bin:" . $ENV{'PATH'};
     open2(\*OUT, \*IN, "/usr/bin/ruby", "-I", "PrettyPatch", "PrettyPatch/prettify.rb", "--html-exceptions");
     $ENV{'PATH'} = $orig_path;
-    print IN $attachment->data . "\n";
+    print IN $attachment->data;
     close(IN);
     while (<OUT>) {
         print;