{"thread":{"id":"32248","subject":"[Query] Can we ignore case for commiters name in shortlog?","startedAt":"2012-11-30T02:09:03Z","lastAt":"2012-11-30T15:05:47Z","messageCount":9,"participants":["viresh kumar","Viresh Kumar","Nicolas Pitre","David Aguilar","Max Horn","Damien Robert","Andreas Schwab"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"204316","messageId":"CAOh2x==NBeeoE2=PhaDC143ZF_xHKD5m=Po+-DS2X43CEeGiEQ@mail.gmail.com","threadId":"32248","inReplyTo":null,"subject":"[Query] Can we ignore case for commiters name in shortlog?","fromName":"viresh kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2012-11-30T02:09:03Z","receivedAt":"2012-11-30T02:09:03Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"Hi Junio and others,\n\nI have a query. git shortlog lists the patches submitted per commiter, like:\n\n> Viresh Kumar (7):\n> cpufreq: Improve debug prints\n> cpufreq: return early from __cpufreq_driver_getavg()\n> cpufreq: governors: remove redundant code\n> cpufreq: Fix sparse warnings by updating cputime64_t to u64\n> cpufreq: Fix sparse warning by making local function static\n> cpufreq: Avoid calling cpufreq driver's target() routine if target_freq == policy->cur\n> cpufreq: Make sure target freq is within limits\n\n> viresh kumar (3):\n> cpufreq / core: Fix typo in comment describing show_bios_limit()\n> cpufreq / core: Fix printing of governor and driver name\n> cpufreq: Move common part from governors to separate file, v2\n\nI know, there was something wrong at my end and i have commited stuff\nwith different cases.\n\nI was just thinking if we can ignore case for commiter name while\nlisting stuff here?\nSo, that we get over any manual mistakes from commiter.\n\n--\nviresh\n"},{"id":"204324","messageId":"CAJDDKr7yr2JSutcEy1mz-SfMq8ZdNzR3+s++ooenn5+wD-LDAw@mail.gmail.com","threadId":"32248","inReplyTo":"CAOh2x==NBeeoE2=PhaDC143ZF_xHKD5m=Po+-DS2X43CEeGiEQ@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2012-11-30T03:24:26Z","receivedAt":"2012-11-30T03:24:26Z","isPatch":false,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"On Thu, Nov 29, 2012 at 6:09 PM, viresh kumar wrote:\n> Hi Junio and others,\n>\n> I have a query. git shortlog lists the patches submitted per commiter, like:\n>\n>> Viresh Kumar (7):\n>> cpufreq: Improve debug prints\n>> cpufreq: return early from __cpufreq_driver_getavg()\n>> cpufreq: governors: remove redundant code\n>> cpufreq: Fix sparse warnings by updating cputime64_t to u64\n>> cpufreq: Fix sparse warning by making local function static\n>> cpufreq: Avoid calling cpufreq driver's target() routine if target_freq == policy->cur\n>> cpufreq: Make sure target freq is within limits\n>\n>> viresh kumar (3):\n>> cpufreq / core: Fix typo in comment describing show_bios_limit()\n>> cpufreq / core: Fix printing of governor and driver name\n>> cpufreq: Move common part from governors to separate file, v2\n>\n> I know, there was something wrong at my end and i have commited stuff\n> with different cases.\n>\n> I was just thinking if we can ignore case for commiter name while\n> listing stuff here?\n> So, that we get over any manual mistakes from commiter.\n\nThere's a feature that does exactly this.\n\nhttp://www.kernel.org/pub/software/scm/git/docs/git-shortlog.html\n\nSee the section called \"Mapping Authors\".\nIt discusses the .mailmap file.\n-- \nDavid\n"},{"id":"204333","messageId":"CAKohpom6PBbC+vp9X4o3ioo1PS9kRbHKiWuMxuR1WeGOeA7L5Q@mail.gmail.com","threadId":"32248","inReplyTo":"CAJDDKr7yr2JSutcEy1mz-SfMq8ZdNzR3+s++ooenn5+wD-LDAw@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2012-11-30T03:32:15Z","receivedAt":"2012-11-30T03:32:15Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 30 November 2012 08:54, David Aguilar wrote:\n> There's a feature that does exactly this.\n>\n> http://www.kernel.org/pub/software/scm/git/docs/git-shortlog.html\n>\n> See the section called \"Mapping Authors\".\n> It discusses the .mailmap file.\n\nI have my name there :)\n\nI thought using names with different case is actually different then misspelling\nit. And so, everybody must not be required to update their names in mailmap\nwith different case. So, with same email id and same name (that may be in\ndifferent case), we can show commits together in shortlog.\n\n--\nviresh\n"},{"id":"204322","messageId":"alpine.LFD.2.02.1211292231520.4576@xanadu.home","threadId":"32248","inReplyTo":"CAOh2x==NBeeoE2=PhaDC143ZF_xHKD5m=Po+-DS2X43CEeGiEQ@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Nicolas Pitre","fromEmail":"nicolas.pitre@linaro.org","sentAt":"2012-11-30T03:33:56Z","receivedAt":"2012-11-30T03:33:56Z","isPatch":false,"sender":{"key":"nicolas.pitre@linaro.org","avatar":null},"body":"On Fri, 30 Nov 2012, viresh kumar wrote:\n\n> Hi Junio and others,\n> \n> I have a query. git shortlog lists the patches submitted per commiter, like:\n> \n> > Viresh Kumar (7):\n> > cpufreq: Improve debug prints\n> > cpufreq: return early from __cpufreq_driver_getavg()\n> > cpufreq: governors: remove redundant code\n> > cpufreq: Fix sparse warnings by updating cputime64_t to u64\n> > cpufreq: Fix sparse warning by making local function static\n> > cpufreq: Avoid calling cpufreq driver's target() routine if target_freq == policy->cur\n> > cpufreq: Make sure target freq is within limits\n> \n> > viresh kumar (3):\n> > cpufreq / core: Fix typo in comment describing show_bios_limit()\n> > cpufreq / core: Fix printing of governor and driver name\n> > cpufreq: Move common part from governors to separate file, v2\n> \n> I know, there was something wrong at my end and i have commited stuff\n> with different cases.\n> \n> I was just thinking if we can ignore case for commiter name while\n> listing stuff here?\n> So, that we get over any manual mistakes from commiter.\n\nHave a look at the .mailmap file in the top directory of your repo.\n\n\nNicolas\n"},{"id":"204317","messageId":"CAKohponGXq=P88Y=cmUPbRCeJ--VkMvJyw5th1T-WdGQnn4xWg@mail.gmail.com","threadId":"32248","inReplyTo":"alpine.LFD.2.02.1211292231520.4576@xanadu.home","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2012-11-30T03:35:26Z","receivedAt":"2012-11-30T03:35:26Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 30 November 2012 09:03, Nicolas Pitre wrote:\n\n> Have a look at the .mailmap file in the top directory of your repo.\n\nRepeating what i said to David in other mail:\n\nI have my name there :)\n\nI thought using names with different case is actually different then misspelling\nit. And so, everybody must not be required to update their names in mailmap\nwith different case. So, with same email id and same name (that may be in\ndifferent case), we can show commits together in shortlog.\n\n--\nviresh\n"},{"id":"204328","messageId":"C9C7BAF7-D552-41CF-B45D-413B9B69C054@quendi.de","threadId":"32248","inReplyTo":"CAKohponGXq=P88Y=cmUPbRCeJ--VkMvJyw5th1T-WdGQnn4xWg@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Max Horn","fromEmail":"max@quendi.de","sentAt":"2012-11-30T11:19:00Z","receivedAt":"2012-11-30T11:19:00Z","isPatch":false,"sender":{"key":"max@quendi.de","avatar":"https://avatars.githubusercontent.com/u/241512?v=4"},"body":"\nOn 30.11.2012, at 04:35, viresh kumar wrote:\n\n> On 30 November 2012 09:03, Nicolas Pitre wrote:\n> \n>> Have a look at the .mailmap file in the top directory of your repo.\n> \n> Repeating what i said to David in other mail:\n> \n> I have my name there :)\n> \n> I thought using names with different case is actually different then misspelling\n> it. And so, everybody must not be required to update their names in mailmap\n> with different case. So, with same email id and same name (that may be in\n> different case), we can show commits together in shortlog.\n\nI don't see how wrong case is different from any other form of misspelling. And mailmap is there precisely to handle such problems. Now, if these case issues were for some reasons very frequent, it might be worth adding dedicated support for it. But this seems dubious to me -- do you have any evidence for this? Indeed, do you have more than just the one example?\n\nIn a nutshell, there seem to be two options here, and I know which *I* find more appealing ;)\n\n1) continue this discussion over several emails to design a new feature (an option, config setting, whatever) to handle your special case, then sit down and write code for it, add documentation, add test cases to test it.\n\n2) You just add a single entry to your mailmap to solve your problem at hand. :-)\n\n\nCheers,\nMax"},{"id":"204335","messageId":"k9aao3$til$1@ger.gmane.org","threadId":"32248","inReplyTo":"CAJDDKr7yr2JSutcEy1mz-SfMq8ZdNzR3+s++ooenn5+wD-LDAw@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Damien Robert","fromEmail":"damien.olivier.robert+gmane@gmail.com","sentAt":"2012-11-30T13:01:55Z","receivedAt":"2012-11-30T13:01:55Z","isPatch":false,"sender":{"key":"damien.olivier.robert+gmane@gmail.com","avatar":null},"body":"David Aguilar wrote in message\n:\n> There's a feature that does exactly this.\n> http://www.kernel.org/pub/software/scm/git/docs/git-shortlog.html\n\nBy the way, the mailmap ignore case which is annoying.\nI have commits as Damien.Olivier.Robert+git@gmail.com and a dummy email\naddress robert@numenor.night-elves. I thought that putting:\n\nDamien Robert \n\nin the .mailmap would unify the two adresses, but it does not:\n\ngit shortlog -se\n 15 Damien Robert \n 266 Damien Robert \n\nas you can see, the Damien.Olivier.Robert+git as been lowercased to\ndamien.olivier.robert, so I am forced to write a mailmap like this:\n\nDamien Robert \nDamien Robert \n\ngit shortlog -se\n 281 Damien Robert \n"},{"id":"204336","messageId":"m27gp3jl2z.fsf@igel.home","threadId":"32248","inReplyTo":"CAOh2x==NBeeoE2=PhaDC143ZF_xHKD5m=Po+-DS2X43CEeGiEQ@mail.gmail.com","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Andreas Schwab","fromEmail":"schwab@linux-m68k.org","sentAt":"2012-11-30T13:07:16Z","receivedAt":"2012-11-30T13:07:16Z","isPatch":false,"sender":{"key":"schwab@linux-m68k.org","avatar":"https://avatars.githubusercontent.com/u/2175493?v=4"},"body":"viresh kumar writes:\n\n> I was just thinking if we can ignore case for commiter name while\n> listing stuff here?\n> So, that we get over any manual mistakes from commiter.\n\nSee git-shortlog(1), section Mapping Authors.\n\nAndreas.\n\n-- \nAndreas Schwab, schwab@linux-m68k.org\nGPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5\n\"And now for something completely different.\"\n"},{"id":"204341","messageId":"CAKohpomGKdeZ6d6_OWYrORy+Cg0yX570jLed9wxW7d+Weytuqw@mail.gmail.com","threadId":"32248","inReplyTo":"C9C7BAF7-D552-41CF-B45D-413B9B69C054@quendi.de","subject":"Re: [Query] Can we ignore case for commiters name in shortlog?","fromName":"Viresh Kumar","fromEmail":"viresh.kumar@linaro.org","sentAt":"2012-11-30T15:05:47Z","receivedAt":"2012-11-30T15:05:47Z","isPatch":false,"sender":{"key":"viresh.kumar@linaro.org","avatar":"https://gravatar.com/avatar/938b1abce576d6c6d2734c01fce1c75040618c8fc0d1b7e836277ecd9f35fb19?d=mp&s=160"},"body":"On 30 November 2012 16:49, Max Horn wrote:\n> I don't see how wrong case is different from any other form of misspelling. And mailmap is there precisely to handle such problems. Now, if these case issues were for some reasons very frequent, it might be worth adding dedicated support for it. But this seems dubious to me -- do you have any evidence for this? Indeed, do you have more than just the one example?\n\nI don't have another example, but i have seen it many times. This happens\nwhen people use different repo's to send patches and by mistake have mentioned\nnames in different case in them in their local .git/config files.\n\n--\nviresh\n"}]}