Re: [PATCH] Optionally parse author information

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxx, 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@xxxxxxxxxxxxxx>")
 - 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.

> 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@xxxxxxxxxxxxxx>
> 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
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]