On Fri, Nov 02, 2012 at 03:43:24PM +0100, Michael J Gruber wrote: > It seems that our fast-import is super picky with regards to author > names. I've encountered author names like > > Foo Bar<foo....@dev.null> > Foo Bar <foo....@dev.null > foo....@dev.null > > in the self-hosting repo of some other dvcs, and the question is how to > translate them faithfully into a git author name.
It is not just fast-import. Git's author field looks like an rfc822 address, but it's much simpler. It fundamentally does not allow angle brackets in the "name" field, regardless of any quoting. As you noted in your followup, we strip them out if you provide them via GIT_AUTHOR_NAME. I doubt this will change anytime soon due to the compatibility fallout. So it is up to generators of fast-import streams to decide how to encode what they get from another system (you could come up with an encoding scheme that represents angle brackets). > In general, we try to do > > fullotherdvcsname <none@none> > > if the other system's entry does not parse as a git author name, but > fast-import does not accept either of > > Foo Bar<foo....@dev.null> <none@none> > "Foo Bar<foo....@dev.null>" <none@none> > > because of the way it parses for <>. While the above could be easily > turned into > > Foo Bar <foo....@dev.null> > > it would not be a faithful representation of the original commit in the > other dvcs. I'd think that if a remote system has names with angle brackets and email-looking things inside them, we would do better to stick them in the email field rather than putting in a useless <none@none>. The latter should only be used for systems that lack the information. But that is a quality-of-implementation issue for the import scripts (and they may even want to have options, just like git-cvsimport allows mapping cvs usernames into full identities). -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
