git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Added an option to cvsimport to specify email domain

From
ANAlexey Nezhdanov <snake@penza-gsm.ru>
Date
Sep 27, 2005, 04:34 UTC
Message-ID
<200509270834.55486.snake@penza-gsm.ru>
In-Reply-To
<20050914193457.GE2936@pasky.or.cz>

(Re-post. Sorry for any inconvenience.) On September 14, 2005 23:34 Petr Baudis wrote:

Show 34 quoted lines
> Dear diary, on Wed, Sep 07, 2005 at 09:18:03PM CEST, I got a letter
> where Junio C Hamano <junkio@cox.net> told me that...
>
> > David K?.A?Negedal <davidk@lysator.liu.se> 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 <domain> 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:
> >
> >     <author> ==> "A U Thor <author@author.dom>"
> >
> > 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 <email>.
> I actually wanted to send a patch, looked at the cvsimport script and
> got totally scared away (at least for now)... ;-)

git uses cvsps output to determine authorship. Do you think if this problem should be solved on cvsps side? It should be relatively easy IMHO. David, can you add another key to output CVSROOT/users mapping result instead of usernames if available?

-- 
Respectfully
Alexey Nezhdanov
Previous: Petr BaudisNext: David Mansfield
Message 4 of 10 in “Added an option to cvsimport to specify email domain”
  1. Added an option to cvsimport to specify email domainDavid Kågedal, Sep 7, 2005
  2. Junio C HamanoSep 7, 2005
  3. Petr BaudisSep 14, 2005
  4. Alexey NezhdanovSep 27, 2005
  5. David MansfieldSep 27, 2005
  6. Alexey NezhdanovSep 30, 2005
  7. Junio C HamanoSep 30, 2005
  8. Alexey NezhdanovSep 30, 2005
  9. David MansfieldSep 30, 2005
  10. Alexey NezhdanovSep 29, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.