git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 1/4] gitk: Fix crash with --all in non-English locales

From
Takashi Iwai <tiwai@suse.de>
Date
Oct 20, 2015, 12:33 UTC
Message-ID
<1445344384-12762-2-git-send-email-tiwai@suse.de>
In-Reply-To
<1445344384-12762-1-git-send-email-tiwai@suse.de>
When gitk is invoked with --all option in a non-English locale, it
crashes like:
$ LC_ALL="de_DE.UTF-8" gitk --all
Error in startup script: bad menu entry index "Ansicht bearbeiten ..."
    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 "/usr/bin/gitk" line 12442)

The reason is the leftover strings that don't match any longer with the new string containing accelerator mark (&). This patch corrects these strings.

Bugzilla: https://bugzilla.suse.com/show_bug.cgi?id=951153
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
 gitk-git/gitk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gitk-git/gitk b/gitk-git/gitk
index 2028b554f487..fcc606eab735 100755
--- a/gitk-git/gitk
+++ b/gitk-git/gitk
@@ -12452,8 +12452,8 @@ if {$cmdline_files ne {} || $revtreeargs ne {} || $revtreeargscmd ne {}} {
     set viewchanged(1) 0
     set vdatemode(1) 0
     addviewmenu 1
-    .bar.view entryconf [mca "Edit view..."] -state normal
-    .bar.view entryconf [mca "Delete view"] -state normal
+    .bar.view entryconf [mca "&Edit view..."] -state normal
+    .bar.view entryconf [mca "&Delete view"] -state normal
 }
 
 if {[info exists permviews]} {
-- 
2.6.1
Previous: Takashi IwaiNext: Takashi Iwai
Message 2 of 6 in “gitk crash fix and locale updates”
  1. 0/4 gitk crash fix and locale updatesTakashi Iwai, Oct 20, 2015
  2. 1/4 gitk: Fix crash with --all in non-English localesTakashi Iwai, Oct 20, 2015
  3. 2/4 gitk: Update msgid's for menu items with acceleratorTakashi Iwai, Oct 20, 2015
  4. 3/4 gitk: Add accelerators to Japanese localeTakashi Iwai, Oct 20, 2015
  5. 4/4 gitk: Add accelerator to German localeTakashi Iwai, Oct 20, 2015
  6. Paul MackerrasOct 30, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.