[Webkit-unassigned] [Bug 27605] Improve git workflow by populating commit messages with ChangeLog entries.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=27605


Adam Treat <treat at kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #33334|review?                     |review-
               Flag|                            |




--- Comment #3 from Adam Treat <treat at kde.org>  2009-07-23 09:06:52 PDT ---
(From update of attachment 33334)
> diff --git a/WebKitTools/ChangeLog b/WebKitTools/ChangeLog
> index 26eae74..654f679 100644
> --- a/WebKitTools/ChangeLog
> +++ b/WebKitTools/ChangeLog
> @@ -1,3 +1,17 @@
> +2009-07-23  Eli Fidler  <eli.fidler at torchmobile.com>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Improve git workflow by populating commit messages with ChangeLog entries.
> +        https://bugs.webkit.org/show_bug.cgi?id=27605
> +
> +		add --[no-]write option to optionally output new ChangeLog entries to
> +		stdout instead of modifying ChangeLog files
> +
> +		fix Torch Mobile copyright

Please fix the indentation.

> -#   Add command line option to put the ChangeLog into a separate
> -#     file or just spew it out stdout.
> +#   Add command line option to put the ChangeLog into a separate file.

Hmm, so this functionality supposedly existed at one point...

> +    if ($prefixDir eq "top level") {
> +        $sortKey = "";
> +    } elsif ($prefixDir eq "Tools") {
> +        $sortKey = "-, just after top level";
> +    } elsif ($prefixDir eq "WebBrowser") {
> +        $sortKey = lc "WebKit, WebBrowser after";
> +    } elsif ($prefixDir eq "WebCore") {
> +        $sortKey = lc "WebFoundation, WebCore after";

I understand that this logic was taken from commit-log-editor.  It'd be nice
for an explanation of where 'WebBrowser' and 'WebFoundation' come from, but I
suspect only the Apple folks would be able to provide that.  Still, it is good
to match commit-log-editor.

The patch looks good minus the minor nits above.  One semi-ugly thing is that
prepare-ChangeLog also writes to stderr by default a bunch of verbosity.  It is
not important for our use case, but perhaps a header or a line break should be
written to stderr right before we write to stdout to separate the two.

r- for the ChangeLog above.

Cheers,
Adam

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.