[Webkit-unassigned] [Bug 220822] New: The generated commit message have a directory label at the first line rather than the bug's title if ChangeLogs have different bug titles

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 21 13:30:25 PST 2021


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

            Bug ID: 220822
           Summary: The generated commit message have a directory label at
                    the first line rather than the bug's title if
                    ChangeLogs have different bug titles
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: Hironori.Fujii at sony.com

If ChangeLogs have different bug titles, the generated commit
message have a directory label at the first line rather than the
bug's title. This is not Git friendly.
For example, r271701, r271695 and r271690.

Tools/Scripts/webkitpy/common/checkout/checkout.py spawns
'commit-log-editor' Perl script to generate a commit message.
https://github.com/WebKit/WebKit/blob/37b6acb392110b1f3e9db830d4207dadef64dbe0/Tools/Scripts/webkitpy/common/checkout/checkout.py#L140

removeLongestCommonPrefixEndingInNewline detects the longest
common header lines of all ChangeLogs and remove the common part
from them and return the prefix and modified ChangeLogs.
https://github.com/WebKit/WebKit/blob/37b6acb392110b1f3e9db830d4207dadef64dbe0/Tools/Scripts/commit-log-editor#L384
Bug 29190 – commit-log-editor should move common prefixes to the top of the commit log

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210121/7e12737a/attachment-0001.htm>


More information about the webkit-unassigned mailing list