[webkit-dev] Proposal: commit-queue to fix usernames after commit

2010-04-12 Thread Eric Seidel
Currently the commit-queue commits all patches as e...@webkit.org.
Some contributers have raised concerns that the commit queue does not
properly credit them for their changes (i.e. in svn annotate), or
concerns that the commit-queue should not be used by committers for
risk of incorrect attribution.


I spoke at length with Bill Siegrist this afternoon and we have a plan
whereby svn.webkit.org could fix svn:author properties on commit-queue
commits.

The proposal is to modify the SVN post-commit hook to:
1.  Grab the author email out of the incoming commit-message/ChangeLog
2.  Check to see if the author email is listed in the SVN authorization file.
3.  If the author is a committer, modify the svn:author property on
the commit (svn propset --revprop -r $REVISION svn:author
$AUTHOR_EMAIL)
4.  Do all rest of the post-commit hooks.

We believe this plan would avoid any Git repository corruption, or
wrong authors in trac or emails as previous proposals for addressing
this issue may have had.

Unless there are objections from the rest of WebKit, Bill and I will
move forward with investigations in the next few weeks.

I would love to hear your feedback!

-eric
___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev


Re: [webkit-dev] Proposal: commit-queue to fix usernames after commit

2010-04-12 Thread David Kilzer
On Mon, April 12, 2010 at 4:56:51 PM, Eric Seidel wrote:


 I would love to hear your feedback!

Sounds great!

Dave

___
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev