# [patch] list shortlog items in commit order

5 messages from 2005-08-02 to 2005-08-03. Participants: Nicolas Pitre, Junio C Hamano, Jeff Garzik, Linus Torvalds.
Thread: https://gitlist.dev/t/1395

## Nicolas Pitre, 2005-08-02 21:18

Subject: [patch] list shortlog items in commit order
Message-ID: <Pine.LNX.4.63.0508021713230.15300@localhost.localdomain>
URL: https://gitlist.dev/e/Pine.LNX.4.63.0508021713230.15300%40localhost.localdomain

```

The current shortlog list is backward making it look odd.
This reverses it so things appear more logically.

Signed-off-by: Nicolas Pitre <nico@cam.org>

diff --git a/git-shortlog b/git-shortlog
--- a/git-shortlog
+++ b/git-shortlog
@@ -90,7 +90,7 @@ sub shortlog_output {
 
 		# output author's 1-line summaries
 		$obj = $map{$key};
-		foreach $desc (@$obj) {
+		foreach $desc (reverse @$obj) {
 			print "  $desc\n";
 			$n_output++;
 		}

```

## Junio C Hamano, 2005-08-02 22:34

Subject: Re: [patch] list shortlog items in commit order
Message-ID: <7vvf2oklwy.fsf@assigned-by-dhcp.cox.net>
URL: https://gitlist.dev/e/7vvf2oklwy.fsf%40assigned-by-dhcp.cox.net
In-Reply-To: <Pine.LNX.4.63.0508021713230.15300@localhost.localdomain>

```
Nicolas Pitre <nico@cam.org> writes:

> The current shortlog list is backward making it look odd.
> This reverses it so things appear more logically.

Sorry, I do not know how the shortlog looked like in BK days,
but it would be nice to match that order.  I do not have
preference either way myself.

I'll let it simmer and let the list decide, but I think the
kernel folks care the most about this one, so I am asking the
original authors, Linus and Jeff, if they want to go for it, or
veto it.

```

## Nicolas Pitre, 2005-08-02 22:41

Subject: Re: [patch] list shortlog items in commit order
Message-ID: <Pine.LNX.4.63.0508021840070.15300@localhost.localdomain>
URL: https://gitlist.dev/e/Pine.LNX.4.63.0508021840070.15300%40localhost.localdomain
In-Reply-To: <7vvf2oklwy.fsf@assigned-by-dhcp.cox.net>

```
On Tue, 2 Aug 2005, Junio C Hamano wrote:

> Nicolas Pitre <nico@cam.org> writes:
> 
> > The current shortlog list is backward making it look odd.
> > This reverses it so things appear more logically.
> 
> Sorry, I do not know how the shortlog looked like in BK days,
> but it would be nice to match that order.

This is precisely why I made this patch: to make it look like the BK 
shortlog.


Nicolas

```

## Jeff Garzik, 2005-08-03 03:58

Subject: Re: [patch] list shortlog items in commit order
Message-ID: <42F040E1.5000706@pobox.com>
URL: https://gitlist.dev/e/42F040E1.5000706%40pobox.com
In-Reply-To: <7vvf2oklwy.fsf@assigned-by-dhcp.cox.net>

```
Junio C Hamano wrote:
> Nicolas Pitre <nico@cam.org> writes:
> 
> 
>>The current shortlog list is backward making it look odd.
>>This reverses it so things appear more logically.
> 
> 
> Sorry, I do not know how the shortlog looked like in BK days,
> but it would be nice to match that order.  I do not have
> preference either way myself.
> 
> I'll let it simmer and let the list decide, but I think the
> kernel folks care the most about this one, so I am asking the
> original authors, Linus and Jeff, if they want to go for it, or
> veto it.

<shrug>  I don't really care either way.

	Jeff

```

## Linus Torvalds, 2005-08-03 04:59

Subject: Re: [patch] list shortlog items in commit order
Message-ID: <Pine.LNX.4.58.0508022158290.3341@g5.osdl.org>
URL: https://gitlist.dev/e/Pine.LNX.4.58.0508022158290.3341%40g5.osdl.org
In-Reply-To: <42F040E1.5000706@pobox.com>

```


On Tue, 2 Aug 2005, Jeff Garzik wrote:
> 
> <shrug>  I don't really care either way.

I suspect it's mostly the users, not the developers, who care. The core
developers already know what went in, and have git to see it, they don't 
look at the shortlog output. So I suspect it's more important to see if 
there's user feedback one way or the other..

		Linus

```