Summary: | Make bug url more human friendly in ChangeLog | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dean Jackson <dino> | ||||||
Component: | Tools / Tests | Assignee: | Dean Jackson <dino> | ||||||
Status: | REOPENED --- | ||||||||
Severity: | Normal | CC: | joepeck, ossy, rniwa, thorton | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 152898 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Dean Jackson
2013-08-13 13:40:52 PDT
I would even prefer a single line for the bug and bug title, like radars: <http://webkit.org/b/119766> Make bug url more human friendly in ChangeLog That would also make `git log --oneline` history for WebKit way easier to read then the current mess. Created attachment 208671 [details]
Patch
OK. Ignore this patch then. BTW, the patch was created with the changes, so webkit-patch works fine. Created attachment 208673 [details]
Patch
Comment on attachment 208673 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208673&action=review > Tools/Scripts/prepare-ChangeLog:198 > + my $bugReadableURL; readableBugURL would be more readable than bugReadableURL. Committed r154022: <http://trac.webkit.org/changeset/154022> This patch broke webkitbot's rollout ability, as it entirely failed to update changelog.py's changelog parsing code. I'm going to fix it in https://bugs.webkit.org/show_bug.cgi?id=120131. Rolled out in http://trac.webkit.org/changeset/154406. There are just too many tools that rely on the old format. Using wekbit.org/b/<~> URL is good but wrapping them in <> and putting the URL and bug title isn't. It makes the long way too long for some bugs and wrapping it in < > breaks all sorts of tools. |