[webkit-dev] Changes to prepare-ChangeLog

[Sam Weinig]

While clearly a personal preference, I don't like the default
prepare-changelog default text and would much rather we revert to the old
one.  I don't think the new text adds anything and will likely be ignored by
most.  Can we leave the --bug= option and revert to the old template?
-Sam


On Wed, Jul 1, 2009 at 10:44 PM, Eric Seidel <eric at webkit.org> wrote:

> Background: Reviewers spend too much time correcting errors which
> should be caught by tools.  This is frustrating both to contributers
> and reviewers.  One such class of errors are missing or incorrect
> information in ChangeLogs.  I'm trying to fix that.
>
>
> As part of:
> http://trac.webkit.org/changeset/45464
>
> prepare-ChangeLog now takes an optional --bug= argument and is able to
> fill in more than before:
>
> % prepare-ChangeLog --bug=26383
>  Running status to find changed, added, or removed files.
>  Reviewing diff to determine which lines changed.
>  Change author: Eric Seidel <eric at webkit.org>.
>  Description from bug 26383:
>    "prepare-ChangeLog should have a --bug= argument and use it for
> url autofill".
>  Editing the ../WebCore/ChangeLog file.
>
> Results in:
> 2009-07-01  Eric Seidel  <eric at webkit.org>
>
>        Reviewed by NOBODY (OOPS!).
>
>        prepare-ChangeLog should have a --bug= argument and use it for
> url autofill
>        https://bugs.webkit.org/show_bug.cgi?id=26383
>
>        DETAILED DESCRIPTION OF THE CHANGES GOES HERE. (OOPS!) SEE:
>        http://webkit.org/coding/contributing.html FOR MORE INFORMATION
>
>        Tests: fast/foo.html
>
>        * foo.cpp: Added.
>
>
>
> Running prepare-ChangeLog by default will however output more boiler-plate:
>
> 2009-07-01  Eric Seidel  <eric at webkit.org>
>
>        Reviewed by NOBODY (OOPS!).
>
>        SHORT DESCRIPTION/BUG TITLE GOES HERE (OOPS!)
>        BUG URL GOES HERE (pass --bug= to autofill)
>
>        DETAILED DESCRIPTION OF THE CHANGES GOES HERE. (OOPS!) SEE:
>        http://webkit.org/coding/contributing.html FOR MORE INFORMATION
>
>        LIST OF TESTS, OR EXPLANATION WHY TESTING IS IMPOSSIBLE GOES
> HERE (OOPS!)
>
>        * foo.cpp: Added.
>
>
> However, hopefully the boilerplate is more helpful than before.
>
>
> Unfortunately everyone will now see:
>        DETAILED DESCRIPTION OF THE CHANGES GOES HERE. (OOPS!) SEE:
>        http://webkit.org/coding/contributing.html FOR MORE INFORMATION
>
> in their initial ChangeLogs.  This may be annoying to seasoned
> contributers (and I'm open to removing it).  But hopefully it will
> lead to better ChangeLogs overall.
>
>
> Looking forward to your comments!
>
> -eric
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090702/0438b81c/attachment.html>