Re: [PATCH] Optionally parse author information
- From
Jonathan Nieder <jrnieder@gmail.com>
- Date
- Jan 16, 2011, 02:57 UTC
- Message-ID
- <20110116025707.GB28452@burratino>
- In-Reply-To
- <AANLkTi=ddJYT8YiUDYy80xobkxJnvuREN-09=464P_vB@mail.gmail.com>
Joe Corneli wrote:
> I tested it, and it seems to use email handle instead of author name > (perhaps that's intentional, though in my case it's not so desirable)
Good point. Presumably git2svn is using the local part of the email address to mimic svn's default behavior of using one's username.
Other possibilities:
- email address (e.g., majordomo@planetmath.org, as in most google repositories) - display name (e.g., "Joe Corneli"). I don't know if svn-related tools or scripts assume that svn:author doesn't contain spaces. - full ident string (e.g., "Joe Corneli <majordomo@planetmath.org>") - whatever the operator wants (mapping specified in authors file).
My guess: an "authors file" facility would be needed to cover all cases, but whichever rule you want to implement short of that could also be useful.
Show 5 quoted lines
> but, quite critically, it gets the dates wrong: > > ~/pmhistory.svn$ svn log -l 5 > ------------------------------------------------------------------------ > r53127 | majordomo | 2011-01-10 18:31:58 -0500 (Mon, 10 Jan 2011) | 1 line
[...]
> ~/pmhistory.git$ git log -5 > commit 411b8698e494ee12799300611fed0c8029e76ad3 > Author: milogardner <majordomo@planetmath.org> > Date: Thu Dec 16 14:11:57 2010 +0000
Maybe it is using the committer date (as shown by "git log --format=fuller") and someone rebased recently. If you don't care about svn's '{date}' construct working (meaning out-of-order dates are ok) then author date might be more suitable. Presumably the important thing is for it to be consistent.
Hope that helps, Jonathan