Bug 57579 - webkit-patch is too strict about bug URL formatting
Summary: webkit-patch is too strict about bug URL formatting
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Adam Roben (:aroben)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-31 14:22 PDT by Adam Roben (:aroben)
Modified: 2011-04-04 11:35 PDT (History)
5 users (show)

See Also:


Attachments
Patch (3.47 KB, patch)
2011-03-31 14:23 PDT, Adam Roben (:aroben)
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Roben (:aroben) 2011-03-31 14:22:18 PDT
webkit-patch is too strict about bug URL formatting
Comment 1 Adam Roben (:aroben) 2011-03-31 14:23:27 PDT
Created attachment 87786 [details]
Patch
Comment 2 Adam Barth 2011-03-31 14:25:05 PDT
Comment on attachment 87786 [details]
Patch

We changed this recently because other folks were sad that we were being too loose.  We should be sure we're making progress here instead of just flip/flopping between what two different camps want.
Comment 3 Adam Barth 2011-03-31 14:25:35 PDT
https://bugs.webkit.org/show_bug.cgi?id=56989 is the earlier bug.
Comment 4 Adam Roben (:aroben) 2011-03-31 14:26:07 PDT
See my comments in bug 56989 for why I think this is justified.
Comment 5 Adam Barth 2011-03-31 14:30:13 PDT
I don't have strong feelings about this topic.
Comment 6 Darin Adler 2011-03-31 15:06:13 PDT
Comment on attachment 87786 [details]
Patch

Seems fine.
Comment 7 Ojan Vafai 2011-03-31 15:50:15 PDT
I'm not totally opposed to this change, but I don't really see the benefit in supporting different changelog formats. IMO, it helps the project if we're all consistent in how we write changelogs.

Some pros of consistency:
-easier to scan changelog entries quickly
-easier to write tooling for dealing with changelog descriptions
-easier for people new to the project to understand them and understand how to write their own descriptions
Comment 8 Adam Roben (:aroben) 2011-03-31 15:55:07 PDT
I agree consistency is a good thing. I feel a little guilty every time I write a non-standard ChangeLog. I was just surprised that the tools started encouraging the format so strongly. And, as stated in bug 56989 comment 15, it seems like a bug that webkit-patch can no longer figure out a bug number for r81850.
Comment 9 Ojan Vafai 2011-03-31 16:18:26 PDT
(In reply to comment #8)
> And, as stated in bug 56989 comment 15, it seems like a bug that webkit-patch can no longer figure out a bug number for r81850.

I guess this is the bit I don't agree with. The bug is only used to update/close the bug. But the patch wasn't actually fixing the bug. The bug is relevant to understanding the patch, but webkit-patch shouldn't be updating the bug.
Comment 10 Adam Roben (:aroben) 2011-03-31 16:31:33 PDT
(In reply to comment #9)
> (In reply to comment #8)
> > And, as stated in bug 56989 comment 15, it seems like a bug that webkit-patch can no longer figure out a bug number for r81850.
> 
> I guess this is the bit I don't agree with. The bug is only used to update/close the bug. But the patch wasn't actually fixing the bug. The bug is relevant to understanding the patch, but webkit-patch shouldn't be updating the bug.

In that case --no-close seems appropriate. It would still be helpful to have webkit-patch add a comment to the bug even if you aren't closing it.
Comment 11 Adam Roben (:aroben) 2011-04-01 08:12:42 PDT
Given that Darin and Eric seem to agree this patch is an improvement (see Eric's comment in bug 56989 comment 16) and that other people are running into trouble because of this (c.f. bug 57605), I'm going to land this patch. We can keep discussing further, but it seems good to make stylebot etc. stop complaining about ChangeLog formats that have long been in use (and worked just fine until this week).
Comment 12 Adam Roben (:aroben) 2011-04-01 08:13:55 PDT
Committed r82681: <http://trac.webkit.org/changeset/82681>
Comment 13 Ojan Vafai 2011-04-01 19:36:08 PDT
(In reply to comment #11)
> Given that Darin and Eric seem to agree this patch is an improvement (see Eric's comment in bug 56989 comment 16) and that other people are running into trouble because of this (c.f. bug 57605), I'm going to land this patch. We can keep discussing further, but it seems good to make stylebot etc. stop complaining about ChangeLog formats that have long been in use (and worked just fine until this week).

Sorry, I though the patch had already been committed. I was just trying to understand what makes sense going forward.
Comment 14 Adam Roben (:aroben) 2011-04-04 11:35:31 PDT
Don't worry, I didn't think you were trying to hold things up. :-) And I'm glad you want to improve the tools, too.