Bug 56416

Summary: Add script to convert a patch to a prettypatch and show in the default browser
Product: WebKit Reporter: Sam Weinig <sam>
Component: New BugsAssignee: Sam Weinig <sam>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch darin: review+

Sam Weinig
Reported 2011-03-15 14:34:07 PDT
Add script to convert a patch to a prettypatch and show in the default browser
Attachments
Patch (3.86 KB, patch)
2011-03-15 14:35 PDT, Sam Weinig 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2011-03-15 14:35:22 PDT
Created attachment 85860 [details] Patch
Darin Adler
Comment 2 2011-03-15 15:56:43 PDT
Comment on attachment 85860 [details] Patch Calling it pretty-diff makes it sound like a diff tool. Maybe you could name it some way that makes it clear that its input is a “patch” or “diff” rather than making it seem like the tool is a “diff” tool.
Sam Weinig
Comment 3 2011-03-15 18:20:06 PDT
Landed as show-pretty-diff in r81206.
WebKit Review Bot
Comment 4 2011-03-15 23:57:53 PDT
http://trac.webkit.org/changeset/81206 might have broken WinCE Release (Build)
Eric Seidel (no email)
Comment 5 2011-03-16 00:04:19 PDT
webkit-patch pretty-diff already knows how to show the current diff as a pretty diff. But it's always good to have more tools. :)
Note You need to log in before you can comment on or make changes to this bug.