# [gitk PATCH] gitk: Disable log.decorate config

4 messages from 2010-03-19 to 2010-03-20. Participants: Santi Béjar, Paul Mackerras, Jeff King.
Thread: https://gitlist.dev/t/23093

## Santi Béjar, 2010-03-19 22:44

Subject: [gitk PATCH] gitk: Disable log.decorate config
Message-ID: <1269038663-11025-1-git-send-email-santi@agolina.net>
URL: https://gitlist.dev/e/1269038663-11025-1-git-send-email-santi%40agolina.net

```
Since eb73445 (Add `log.decorate' configuration variable., 2010-02-17)
the log output changes if log.decorate is set. Gitk is unable to
understand the new output (it returns the error: "Can't parse git log
output: {commit $sha1 $sha2...}), so disable it with --no-decorate.

Signed-off-by: Santi Béjar <santi@agolina.net>
---
 gitk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/gitk b/gitk
index 1f36a3e..aca44b7 100755
--- a/gitk
+++ b/gitk
@@ -362,7 +362,7 @@ proc start_rev_list {view} {
 
     if {[catch {
 	set fd [open [concat | git log --no-color -z --pretty=raw --parents \
-			 --boundary $args "--" $files] r]
+			 --boundary --no-decorate $args "--" $files] r]
     } err]} {
 	error_popup "[mc "Error executing git log:"] $err"
 	return 0
-- 
1.7.0.2.384.g015e

```

## Paul Mackerras, 2010-03-20 05:10

Subject: Re: [gitk PATCH] gitk: Disable log.decorate config
Message-ID: <20100320051055.GA3855@brick.ozlabs.ibm.com>
URL: https://gitlist.dev/e/20100320051055.GA3855%40brick.ozlabs.ibm.com
In-Reply-To: <1269038663-11025-1-git-send-email-santi@agolina.net>

```
On Fri, Mar 19, 2010 at 11:44:23PM +0100, Santi Béjar wrote:

> Since eb73445 (Add `log.decorate' configuration variable., 2010-02-17)
> the log output changes if log.decorate is set. Gitk is unable to
> understand the new output (it returns the error: "Can't parse git log
> output: {commit $sha1 $sha2...}), so disable it with --no-decorate.
> 
> Signed-off-by: Santi Béjar <santi@agolina.net>
> ---
>  gitk |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/gitk b/gitk
> index 1f36a3e..aca44b7 100755
> --- a/gitk
> +++ b/gitk
> @@ -362,7 +362,7 @@ proc start_rev_list {view} {
>  
>      if {[catch {
>  	set fd [open [concat | git log --no-color -z --pretty=raw --parents \
> -			 --boundary $args "--" $files] r]
> +			 --boundary --no-decorate $args "--" $files] r]

We'll need to make that conditional on the git version, since we want
gitk to continue to work on older git installations.

Paul.

```

## Jeff King, 2010-03-20 05:34

Subject: Re: [gitk PATCH] gitk: Disable log.decorate config
Message-ID: <20100320053404.GA12625@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20100320053404.GA12625%40coredump.intra.peff.net
In-Reply-To: <1269038663-11025-1-git-send-email-santi@agolina.net>

```
On Fri, Mar 19, 2010 at 11:44:23PM +0100, Santi Béjar wrote:

> Since eb73445 (Add `log.decorate' configuration variable., 2010-02-17)
> the log output changes if log.decorate is set. Gitk is unable to
> understand the new output (it returns the error: "Can't parse git log
> output: {commit $sha1 $sha2...}), so disable it with --no-decorate.

log.decorate is still in "next". My understanding was that it was not
going to graduate to "master" until this issue had been resolved, and
that the current plan was to disable config decoration for --pretty=raw.
That would solve the issue without any effort on the part of gitk.

-Peff

```

## Santi Béjar, 2010-03-20 11:39

Subject: Re: [gitk PATCH] gitk: Disable log.decorate config
Message-ID: <adf1fd3d1003200439o52d45632v7096b6bef64f744b@mail.gmail.com>
URL: https://gitlist.dev/e/adf1fd3d1003200439o52d45632v7096b6bef64f744b%40mail.gmail.com
In-Reply-To: <20100320051055.GA3855@brick.ozlabs.ibm.com>

```
On Sat, Mar 20, 2010 at 6:10 AM, Paul Mackerras <paulus@samba.org> wrote:
> On Fri, Mar 19, 2010 at 11:44:23PM +0100, Santi Béjar wrote:
>
>> Since eb73445 (Add `log.decorate' configuration variable., 2010-02-17)
>> the log output changes if log.decorate is set. Gitk is unable to
>> understand the new output (it returns the error: "Can't parse git log
>> output: {commit $sha1 $sha2...}), so disable it with --no-decorate.
>>
>> Signed-off-by: Santi Béjar <santi@agolina.net>
>> ---
>>  gitk |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/gitk b/gitk
>> index 1f36a3e..aca44b7 100755
>> --- a/gitk
>> +++ b/gitk
>> @@ -362,7 +362,7 @@ proc start_rev_list {view} {
>>
>>      if {[catch {
>>       set fd [open [concat | git log --no-color -z --pretty=raw --parents \
>> -                      --boundary $args "--" $files] r]
>> +                      --boundary --no-decorate $args "--" $files] r]
>
> We'll need to make that conditional on the git version, since we want
> gitk to continue to work on older git installations.

I see. I didn't find a version check for the --no-color (I know it is
much older, but still), so I did the same.

But as Jeff said it will be solved within git itself.

Santi

```