I think I've complained about this in the past, but can't find the mail. Dear diary, on Thu, Oct 26, 2006 at 11:25:50AM CEST, I got a letter where Jakub Narebski <jnareb@xxxxxxxxx> said that... > Johannes Schindelin wrote: > > > BTW what happened to the builtin shortlog? It is the last Perl script I > > use regularly... (should make people happy who are stuck with Activision > > Perl...) > > BTW. both Perl version and builtin shorlog have email->real name translation > table built in. In Perl script version it is in __DATA__ section, and we > could update it using Inline::Files module, in C version it was in table. > But in fact this list is project specific. Shouldn't we make it customizable > (::sigh::, yet another file in $GIT_DIR...). I really dislike the fact that we _do_ this mapping at all, this seems so much a totally wrong point at which to do it. The information tracked in Git is still wrong and all the tools except shortlog still display it wrong - why should shortlog in particular be special? Why don't we do this at the git-am time instead? And overally, things would be simpler if people would just require the author name to be recorded in the mail properly when applying the patch; AIUI at least in case of the kernel this mapping shouldn't really be needed anymore anyway since with the signoff protocol, you already require all the contributors to reveal their realnames in signoffs? People can then as well just write that in their from headers as well. Linus? So what about making git-am by default refuse to apply patches with missing author realnames? -- Petr "Pasky" Baudis Stuff: http://pasky.or.cz/ #!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj $/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1 lK[d2%Sa2/d0$^Ixp"|dc`;s/\W//g;$_=pack('H*',/((..)*)$/) - 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