# [PATCH] gitweb: Remove redundant "commit" link from shortlog

5 messages from 2006-09-28 to 2006-10-06. Participants: Luben Tuikov, Petr Baudis, Jakub Narebski.
Thread: https://gitlist.dev/t/5750

## Luben Tuikov, 2006-09-28 23:48

Subject: [PATCH] gitweb: Remove redundant "commit" link from shortlog
Message-ID: <20060928234840.25950.qmail@web31809.mail.mud.yahoo.com>
URL: https://gitlist.dev/e/20060928234840.25950.qmail%40web31809.mail.mud.yahoo.com

```
Remove the redundant "commit" link from shortlog.
It can be had by simply clicking on the entry title
of the row.

Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
---
 gitweb/gitweb.perl |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)


diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 79c8052..f3f22c2 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -2013,7 +2013,6 @@ sub git_shortlog_body {
 		                          href(action=>"commit", hash=>$commit), $ref);
 		print "</td>\n" .
 		      "<td class=\"link\">" .
-		      $cgi->a({-href => href(action=>"commit", hash=>$commit)}, "commit") . " | " .
 		      $cgi->a({-href => href(action=>"commitdiff", hash=>$commit)}, "commitdiff") . " | " .
 		      $cgi->a({-href => href(action=>"tree", hash=>$commit, hash_base=>$commit)}, "tree");
 		print "</td>\n" .
-- 
1.4.2.1.g05f0f


```

## Petr Baudis, 2006-10-06 13:55

Subject: Re: [PATCH] gitweb: Remove redundant "commit" link from shortlog
Message-ID: <20061006135508.GO20017@pasky.or.cz>
URL: https://gitlist.dev/e/20061006135508.GO20017%40pasky.or.cz
In-Reply-To: <20060928234840.25950.qmail@web31809.mail.mud.yahoo.com>

```
Dear diary, on Fri, Sep 29, 2006 at 01:48:40AM CEST, I got a letter
where Luben Tuikov <ltuikov@yahoo.com> said that...
> Remove the redundant "commit" link from shortlog.
> It can be had by simply clicking on the entry title
> of the row.
> 
> Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>

And what if the commit message is empty?

Witness at http://repo.or.cz/?p=test;a=summary

-- 
				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*',/((..)*)$/)

```

## Jakub Narebski, 2006-10-06 15:08

Subject: Re: [PATCH] gitweb: Remove redundant "commit" link from shortlog
Message-ID: <eg5rgf$58c$2@sea.gmane.org>
URL: https://gitlist.dev/e/eg5rgf%2458c%242%40sea.gmane.org
In-Reply-To: <20061006135508.GO20017@pasky.or.cz>

```
Petr Baudis wrote:

> Dear diary, on Fri, Sep 29, 2006 at 01:48:40AM CEST, I got a letter
> where Luben Tuikov <ltuikov@yahoo.com> said that...
>> Remove the redundant "commit" link from shortlog.
>> It can be had by simply clicking on the entry title
>> of the row.
>> 
>> Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
> 
> And what if the commit message is empty?
> 
> Witness at http://repo.or.cz/?p=test;a=summary

By the way, commitdiff view in this case also looks strange.
I think that simplyfying the git_difftree_body went too far,
Luben.
-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git

```

## Petr Baudis, 2006-10-06 16:55

Subject: [PATCH] gitweb: Handle commits with empty commit messages more reasonably
Message-ID: <20061006165504.3934.97392.stgit@rover>
URL: https://gitlist.dev/e/20061006165504.3934.97392.stgit%40rover
In-Reply-To: <20061006135508.GO20017@pasky.or.cz>

```
Currently those look very weird, you can't get easily at the commit view
etc. This patch makes their title '(no commit message)'.

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 gitweb/gitweb.perl |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index ea159e9..56e5231 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -1115,6 +1115,9 @@ sub parse_commit {
 			last;
 		}
 	}
+	if ($co{'title'} eq "") {
+		$co{'title'} = $co{'title_short'} = '(no commit message)';
+	}
 	# remove added spaces
 	foreach my $line (@commit_lines) {
 		$line =~ s/^    //;

```

## Luben Tuikov, 2006-10-06 18:27

Subject: Re: [PATCH] gitweb: Remove redundant "commit" link from shortlog
Message-ID: <20061006182720.88733.qmail@web31812.mail.mud.yahoo.com>
URL: https://gitlist.dev/e/20061006182720.88733.qmail%40web31812.mail.mud.yahoo.com
In-Reply-To: <20061006135508.GO20017@pasky.or.cz>

```
--- Petr Baudis <pasky@suse.cz> wrote:
> Dear diary, on Fri, Sep 29, 2006 at 01:48:40AM CEST, I got a letter
> where Luben Tuikov <ltuikov@yahoo.com> said that...
> > Remove the redundant "commit" link from shortlog.
> > It can be had by simply clicking on the entry title
> > of the row.
> > 
> > Signed-off-by: Luben Tuikov <ltuikov@yahoo.com>
> 
> And what if the commit message is empty?
> 
> Witness at http://repo.or.cz/?p=test;a=summary

Clearly a pathological case.  Who or why would anyone do this?
I don't think we should worry about such pathological cases.

Commits without commit messages do not question GIT.  They
question the committer... (that's the morning chuckle)

    Luben


> 
> -- 
> 				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@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

```