From: Petr Baudis Date: Wed, 14 Sep 2005 19:34:57 GMT Subject: Re: [PATCH] Added an option to cvsimport to specify email domain Message-ID: <20050914193457.GE2936@pasky.or.cz> In-Reply-To: <7vhdcw661g.fsf@assigned-by-dhcp.cox.net> Dear diary, on Wed, Sep 07, 2005 at 09:18:03PM CEST, I got a letter where Junio C Hamano told me that... > David K?.A?Negedal writes: > > > The authorship info in commits created by git-cvsimport-script > > only contains the username of the CVS committer. This patch > > adds a flag -e to git-cvsimport-script that makes it > > possible to specify an email domain that is added to all email > > addresses in the commit "author" and "committer" fields. > > > > --- > > I have stopped using cvsimport, because cvsps seems to produce bad > > output on the repository I'm using it with, but I had already prepared > > this patch. > > Hmph. One reason the original implementation did not do this is > because Linus and other people wanted to have a repeatability, > so making this an optional thing is good, but if we go this > route, I think if it would be nicer to have a --author-map > option that lets you feed a list of: > > ==> "A U Thor " > > mappings, instead of a single -e, which essentially does not add > much information to the result. > > I take that your oob comment indicates that you do not have much > incentive/inclination to further hack on this, so I am not > asking you to do the above even if you find my suggestion > worthwhile. Various tools use CVSROOT/users to map usernames to realname . I actually wanted to send a patch, looked at the cvsimport script and got totally scared away (at least for now)... ;-) -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ If you want the holes in your knowledge showing up try teaching someone. -- Alan Cox