threads / bug / 40831

[BUG] gitk --all and git bisect visualize crash in non-english locale

Subject: [BUG] gitk --all and git bisect visualize crash in non-english locale

## tl;dr

3 messages between Nov 18, 2015 and Nov 20, 2015.

replies: 2people: 2as markdown or json

Matthieu Moy· Nov 18, 2015, 18:31 UTC · lore
Hi,
I'm getting the following crash on recent gitk:
  $ LANG=fr_FR gitk --all
  Error in startup script: bad menu entry index "Éditer la vue..."
      while executing
  ".bar.view entryconf [mca "Edit view..."] -state normal"
      invoked from within
  "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd ne {}} {
      # create a view for the files/dirs specified on the command line
      se..."
      (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
It's linked to the locale:
  $ LANG=C gitk --all
  # Just works

but does not seem to be related to French in particular nor to the accent:

  $ LANG=es_ES gitk --all
  Error in startup script: bad menu entry index "Modificar vista..."
      while executing
  ".bar.view entryconf [mca "Edit view..."] -state normal"
      invoked from within
  "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd ne {}} {
      # create a view for the files/dirs specified on the command line
      se..."
      (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
It also works fine without --all.
It bisects down to:
commit d99b4b0de27a2bd654a40353b65883e368da6d06
Author: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Date:   Wed Sep 9 15:20:53 2015 +0200
    gitk: Accelerators for the main menu
    
    This allows fast, keyboard-only usage of the menu (e.g. Alt+V, N to open a
    new view).
    
    Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
    Signed-off-by: Paul Mackerras <paulus@samba.org>
I'm not fluent enough in Tcl to fix this myself, sorry ;-).
Thanks,
-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Dennis Kaarsemaker· Nov 18, 2015, 21:53 UTC · re: Matthieu Moy · lore

Re: [BUG] gitk --all and git bisect visualize crash in non-english locale

Hi Matthieu,

This has been reported quite a few times before. A fix is in master, see 482456a^2 and children.

On wo, 2015-11-18 at 19:31 +0100, Matthieu Moy wrote:
Show 57 quoted lines
> Hi,
> 
> I'm getting the following crash on recent gitk:
> 
>   $ LANG=fr_FR gitk --all
>   Error in startup script: bad menu entry index "Éditer la vue..."
>       while executing
>   ".bar.view entryconf [mca "Edit view..."] -state normal"
>       invoked from within
>   "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd
> ne {}} {
>       # create a view for the files/dirs specified on the command
> line
>       se..."
>       (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
> 
> It's linked to the locale:
> 
>   $ LANG=C gitk --all
>   # Just works
> 
> but does not seem to be related to French in particular nor to the
> accent:
> 
>   $ LANG=es_ES gitk --all
>   Error in startup script: bad menu entry index "Modificar vista..."
>       while executing
>   ".bar.view entryconf [mca "Edit view..."] -state normal"
>       invoked from within
>   "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd
> ne {}} {
>       # create a view for the files/dirs specified on the command
> line
>       se..."
>       (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
> 
> It also works fine without --all.
> 
> It bisects down to:
> 
> commit d99b4b0de27a2bd654a40353b65883e368da6d06
> Author: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
> Date:   Wed Sep 9 15:20:53 2015 +0200
> 
>     gitk: Accelerators for the main menu
>     
>     This allows fast, keyboard-only usage of the menu (e.g. Alt+V, N
> to open a
>     new view).
>     
>     Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
>     Signed-off-by: Paul Mackerras <paulus@samba.org>
> 
> I'm not fluent enough in Tcl to fix this myself, sorry ;-).
> 
> Thanks,
> 
-- 
Dennis Kaarsemaker
www.kaarsemaker.net
Matthieu Moy· Nov 20, 2015, 14:49 UTC · re: Dennis Kaarsemaker · lore

Re: [BUG] gitk --all and git bisect visualize crash in non-english locale

Indeed. I should obviously have started by updating my master branch, sorry for the noise.

Thanks,
Dennis Kaarsemaker <dennis@kaarsemaker.net> writes:
Show 63 quoted lines
> Hi Matthieu,
>
> This has been reported quite a few times before. A fix is in master,
> see 482456a^2 and children.
>
> On wo, 2015-11-18 at 19:31 +0100, Matthieu Moy wrote:
>> Hi,
>> 
>> I'm getting the following crash on recent gitk:
>> 
>>   $ LANG=fr_FR gitk --all
>>   Error in startup script: bad menu entry index "Éditer la vue..."
>>       while executing
>>   ".bar.view entryconf [mca "Edit view..."] -state normal"
>>       invoked from within
>>   "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd
>> ne {}} {
>>       # create a view for the files/dirs specified on the command
>> line
>>       se..."
>>       (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
>> 
>> It's linked to the locale:
>> 
>>   $ LANG=C gitk --all
>>   # Just works
>> 
>> but does not seem to be related to French in particular nor to the
>> accent:
>> 
>>   $ LANG=es_ES gitk --all
>>   Error in startup script: bad menu entry index "Modificar vista..."
>>       while executing
>>   ".bar.view entryconf [mca "Edit view..."] -state normal"
>>       invoked from within
>>   "if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd
>> ne {}} {
>>       # create a view for the files/dirs specified on the command
>> line
>>       se..."
>>       (file "/home/moy/local/usr-jessie/bin/gitk" line 12442)
>> 
>> It also works fine without --all.
>> 
>> It bisects down to:
>> 
>> commit d99b4b0de27a2bd654a40353b65883e368da6d06
>> Author: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
>> Date:   Wed Sep 9 15:20:53 2015 +0200
>> 
>>     gitk: Accelerators for the main menu
>>     
>>     This allows fast, keyboard-only usage of the menu (e.g. Alt+V, N
>> to open a
>>     new view).
>>     
>>     Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
>>     Signed-off-by: Paul Mackerras <paulus@samba.org>
>> 
>> I'm not fluent enough in Tcl to fix this myself, sorry ;-).
>> 
>> Thanks,
>> 
-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

← back to recent threads