WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
56416
Add script to convert a patch to a prettypatch and show in the default browser
https://bugs.webkit.org/show_bug.cgi?id=56416
Summary
Add script to convert a patch to a prettypatch and show in the default browser
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug