{"thread":{"id":"1395","subject":"[patch] list shortlog items in commit order","startedAt":"2005-08-02T21:18:57Z","lastAt":"2005-08-03T04:59:24Z","messageCount":5,"participants":["Nicolas Pitre","Junio C Hamano","Jeff Garzik","Linus Torvalds"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"6721","messageId":"Pine.LNX.4.63.0508021713230.15300@localhost.localdomain","threadId":"1395","inReplyTo":null,"subject":"[patch] list shortlog items in commit order","fromName":"Nicolas Pitre","fromEmail":"nico@cam.org","sentAt":"2005-08-02T21:18:57Z","receivedAt":"2005-08-02T21:18:57Z","isPatch":true,"sender":{"key":"nico@fluxnic.net","avatar":"https://avatars.githubusercontent.com/u/702790?v=4"},"body":"\nThe current shortlog list is backward making it look odd.\nThis reverses it so things appear more logically.\n\nSigned-off-by: Nicolas Pitre <nico@cam.org>\n\ndiff --git a/git-shortlog b/git-shortlog\n--- a/git-shortlog\n+++ b/git-shortlog\n@@ -90,7 +90,7 @@ sub shortlog_output {\n \n \t\t# output author's 1-line summaries\n \t\t$obj = $map{$key};\n-\t\tforeach $desc (@$obj) {\n+\t\tforeach $desc (reverse @$obj) {\n \t\t\tprint \"  $desc\\n\";\n \t\t\t$n_output++;\n \t\t}\n"},{"id":"6726","messageId":"7vvf2oklwy.fsf@assigned-by-dhcp.cox.net","threadId":"1395","inReplyTo":"Pine.LNX.4.63.0508021713230.15300@localhost.localdomain","subject":"Re: [patch] list shortlog items in commit order","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2005-08-02T22:34:53Z","receivedAt":"2005-08-02T22:34:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Nicolas Pitre <nico@cam.org> writes:\n\n> The current shortlog list is backward making it look odd.\n> This reverses it so things appear more logically.\n\nSorry, I do not know how the shortlog looked like in BK days,\nbut it would be nice to match that order.  I do not have\npreference either way myself.\n\nI'll let it simmer and let the list decide, but I think the\nkernel folks care the most about this one, so I am asking the\noriginal authors, Linus and Jeff, if they want to go for it, or\nveto it.\n"},{"id":"6729","messageId":"Pine.LNX.4.63.0508021840070.15300@localhost.localdomain","threadId":"1395","inReplyTo":"7vvf2oklwy.fsf@assigned-by-dhcp.cox.net","subject":"Re: [patch] list shortlog items in commit order","fromName":"Nicolas Pitre","fromEmail":"nico@cam.org","sentAt":"2005-08-02T22:41:48Z","receivedAt":"2005-08-02T22:41:48Z","isPatch":true,"sender":{"key":"nico@fluxnic.net","avatar":"https://avatars.githubusercontent.com/u/702790?v=4"},"body":"On Tue, 2 Aug 2005, Junio C Hamano wrote:\n\n> Nicolas Pitre <nico@cam.org> writes:\n> \n> > The current shortlog list is backward making it look odd.\n> > This reverses it so things appear more logically.\n> \n> Sorry, I do not know how the shortlog looked like in BK days,\n> but it would be nice to match that order.\n\nThis is precisely why I made this patch: to make it look like the BK \nshortlog.\n\n\nNicolas\n"},{"id":"6740","messageId":"42F040E1.5000706@pobox.com","threadId":"1395","inReplyTo":"7vvf2oklwy.fsf@assigned-by-dhcp.cox.net","subject":"Re: [patch] list shortlog items in commit order","fromName":"Jeff Garzik","fromEmail":"jgarzik@pobox.com","sentAt":"2005-08-03T03:58:25Z","receivedAt":"2005-08-03T03:58:25Z","isPatch":true,"sender":{"key":"jgarzik@pobox.com","avatar":null},"body":"Junio C Hamano wrote:\n> Nicolas Pitre <nico@cam.org> writes:\n> \n> \n>>The current shortlog list is backward making it look odd.\n>>This reverses it so things appear more logically.\n> \n> \n> Sorry, I do not know how the shortlog looked like in BK days,\n> but it would be nice to match that order.  I do not have\n> preference either way myself.\n> \n> I'll let it simmer and let the list decide, but I think the\n> kernel folks care the most about this one, so I am asking the\n> original authors, Linus and Jeff, if they want to go for it, or\n> veto it.\n\n<shrug>  I don't really care either way.\n\n\tJeff\n"},{"id":"6741","messageId":"Pine.LNX.4.58.0508022158290.3341@g5.osdl.org","threadId":"1395","inReplyTo":"42F040E1.5000706@pobox.com","subject":"Re: [patch] list shortlog items in commit order","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2005-08-03T04:59:24Z","receivedAt":"2005-08-03T04:59:24Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Tue, 2 Aug 2005, Jeff Garzik wrote:\n> \n> <shrug>  I don't really care either way.\n\nI suspect it's mostly the users, not the developers, who care. The core\ndevelopers already know what went in, and have git to see it, they don't \nlook at the shortlog output. So I suspect it's more important to see if \nthere's user feedback one way or the other..\n\n\t\tLinus\n"}]}