Bug 33246 - webkit-patch land should warn when changelog doesn't match committer
Summary: webkit-patch land should warn when changelog doesn't match committer
Status: RESOLVED DUPLICATE of bug 28092
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-05 18:25 PST by Ojan Vafai
Modified: 2010-04-01 23:21 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ojan Vafai 2010-01-05 18:25:53 PST 
I just had the following happen:
1. git svn rebase successfully merged my ChangeLog file, but didn't put my entry at the top
2. bugzilla-tool land-diff committed my change with the wrong commit message

See http://trac.webkit.org/changeset/52844

Technically, I should have noticed in the diff that the ChangeLog entry wasn't at the top, but it would be nice to be a bit more idiot-proof. Can land-diff confirm that you want to commit if the email address doesn't match git's configured email address? Obviously, this would always be true on the commit bot, but for human beings committing, I think this is by far the common case (i.e. that they're committing their own patches).
Comment 1 Eric Seidel (no email) 2010-01-19 17:00:18 PST 
We have like 8 dupes of this bug on file.  Git users run into this kind of thing all the time. :(
Comment 2 Eric Seidel (no email) 2010-04-01 23:21:52 PDT 

*** This bug has been marked as a duplicate of bug 28092 ***