{"thread":{"id":"23000","subject":"[PATCH 1/3] gitk: Avoid calling tk_setPalette on Windows","startedAt":"2010-03-12T18:31:47Z","lastAt":"2010-03-23T02:29:03Z","messageCount":10,"participants":["Pat Thoyts","Jonathan Nieder"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"136680","messageId":"1268418709-4998-1-git-send-email-patthoyts@users.sourceforge.net","threadId":"23000","inReplyTo":null,"subject":"[PATCH 1/3] gitk: Avoid calling tk_setPalette on Windows","fromName":"Pat Thoyts","fromEmail":"patthoyts@users.sourceforge.net","sentAt":"2010-03-12T18:31:47Z","receivedAt":"2010-03-12T18:31:47Z","isPatch":true,"sender":{"key":"patthoyts@users.sourceforge.net","avatar":"https://avatars.githubusercontent.com/u/30739?v=4"},"body":"This just messes up the system colors. Leave them alone.\n\nSigned-off-by: Pat Thoyts \n---\n gitk | 1 +\n 1 files changed, 1 insertions(+), 0 deletions(-)\n\ndiff --git a/gitk b/gitk\nindex 1f36a3e..dc9b8a8 100755\n--- a/gitk\n+++ b/gitk\n@@ -10849,6 +10849,7 @@ proc setselbg {c} {\n # radiobuttons look bad. This chooses white for selectColor if the\n # background color is light, or black if it is dark.\n proc setui {c} {\n+ if {[tk windowingsystem] eq \"win32\"} { return }\n set bg [winfo rgb . $c]\n set selc black\n if {[lindex $bg 0] + 1.5 * [lindex $bg 1] + 0.5 * [lindex $bg 2] > 100000} {\n-- \n1.6.6\n"},{"id":"136682","messageId":"1268418709-4998-2-git-send-email-patthoyts@users.sourceforge.net","threadId":"23000","inReplyTo":"1268418709-4998-1-git-send-email-patthoyts@users.sourceforge.net","subject":"[PATCH 2/3] gitk: Add emacs editor variable block.","fromName":"Pat Thoyts","fromEmail":"patthoyts@users.sourceforge.net","sentAt":"2010-03-12T18:31:48Z","receivedAt":"2010-03-12T18:31:48Z","isPatch":true,"sender":{"key":"patthoyts@users.sourceforge.net","avatar":"https://avatars.githubusercontent.com/u/30739?v=4"},"body":"Help contributors use the correct indentation style.\n\nSigned-off-by: Pat Thoyts \n---\n gitk | 6 ++++++\n 1 files changed, 6 insertions(+), 0 deletions(-)\n\ndiff --git a/gitk b/gitk\nindex dc9b8a8..ed5f6ab 100755\n--- a/gitk\n+++ b/gitk\n@@ -11614,3 +11614,9 @@ if {[tk windowingsystem] eq \"win32\"} {\n }\n \n getcommits {}\n+\n+# Local variables:\n+# mode: tcl\n+# indent-tabs-mode: t\n+# tab-width: 8\n+# End:\n-- \n1.6.6\n"},{"id":"136681","messageId":"1268418709-4998-3-git-send-email-patthoyts@users.sourceforge.net","threadId":"23000","inReplyTo":"1268418709-4998-2-git-send-email-patthoyts@users.sourceforge.net","subject":"[PATCH 3/3] gitk: fix display of copyright symbol","fromName":"Pat Thoyts","fromEmail":"patthoyts@users.sourceforge.net","sentAt":"2010-03-12T18:31:49Z","receivedAt":"2010-03-12T18:31:49Z","isPatch":true,"sender":{"key":"patthoyts@users.sourceforge.net","avatar":"https://avatars.githubusercontent.com/u/30739?v=4"},"body":"The script file uses utf-8 encoding but when sourced it will be read\nusing the default system encoding which is never utf8 on windows. This\ncauses the copyright symbol to display incorrectly in the about dialog.\nUsing the unicode escape sequence avoids incorrect decoding but does\nrequire a double escape in the .po files.\n\nAlso adjusted the year range.\n\nSigned-off-by: Pat Thoyts \n---\n gitk | 2 +-\n po/de.po | 4 ++--\n po/es.po | 4 ++--\n po/fr.po | 4 ++--\n po/hu.po | 4 ++--\n po/it.po | 4 ++--\n po/ja.po | 4 ++--\n po/ru.po | 4 ++--\n po/sv.po | 4 ++--\n 9 files changed, 17 insertions(+), 17 deletions(-)\n\ndiff --git a/gitk b/gitk\nindex 1f36a3e..e1f3740 100755\n--- a/gitk\n+++ b/gitk\n@@ -2782,7 +2782,7 @@ proc about {} {\n message $w.m -text [mc \"\n Gitk - a commit viewer for git\n \n-Copyright © 2005-2009 Paul Mackerras\n+Copyright \\u00a9 2005-2010 Paul Mackerras\n \n Use and redistribute under the terms of the GNU General Public License\"] \\\n \t -justify center -aspect 400 -border 2 -bg white -relief groove\ndiff --git a/po/de.po b/po/de.po\nindex c79aa9c..bd194a3 100644\n--- a/po/de.po\n+++ b/po/de.po\n@@ -334,14 +334,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright ©9 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - eine Visualisierung der Git-Historie\\n\"\n \"\\n\"\n-\"Copyright © 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Benutzung und Weiterverbreitung gemäß den Bedingungen der GNU General Public License\"\n \ndiff --git a/po/es.po b/po/es.po\nindex 0e19b5e..0471dd0 100644\n--- a/po/es.po\n+++ b/po/es.po\n@@ -281,14 +281,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - un visualizador de revisiones para git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Uso y redistribución permitidos según los términos de la Licencia Pública \"\n \"General de GNU (GNU GPL)\"\ndiff --git a/po/fr.po b/po/fr.po\nindex cb0e1ed..5370ddc 100644\n--- a/po/fr.po\n+++ b/po/fr.po\n@@ -334,14 +334,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - visualisateur de commit pour git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Utilisation et redistribution soumises aux termes de la GNU General Public \"\n \"License\"\ndiff --git a/po/hu.po b/po/hu.po\nindex 1df212e..7262b61 100644\n--- a/po/hu.po\n+++ b/po/hu.po\n@@ -333,14 +333,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright ©9 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - commit nézegető a githez\\n\"\n \"\\n\"\n-\"Szerzői jog ©9 2005-2009 Paul Mackerras\\n\"\n+\"Szerzői jog \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Használd és terjeszd a GNU General Public License feltételei mellett\"\n \ndiff --git a/po/it.po b/po/it.po\nindex 4818652..a730d63 100644\n--- a/po/it.po\n+++ b/po/it.po\n@@ -334,14 +334,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - un visualizzatore di revisioni per git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Utilizzo e redistribuzione permessi sotto i termini della GNU General Public \"\n \"License\"\ndiff --git a/po/ja.po b/po/ja.po\nindex c0c92ad..4f47051 100644\n--- a/po/ja.po\n+++ b/po/ja.po\n@@ -335,14 +335,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - gitコミットビューア\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"使用および再配布は GNU General Public License に従ってください\"\n \ndiff --git a/po/ru.po b/po/ru.po\nindex 704eba8..c3d0285 100644\n--- a/po/ru.po\n+++ b/po/ru.po\n@@ -313,14 +313,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - программа просмотра истории репозиториев Git\\n\"\n \"\\n\"\n-\"Copyright (c) 2005-2008 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Использование и распространение согласно условиям GNU General Public License\"\n \ndiff --git a/po/sv.po b/po/sv.po\nindex 0f5e2fd..386763a 100644\n--- a/po/sv.po\n+++ b/po/sv.po\n@@ -334,14 +334,14 @@ msgid \"\"\n \"\\n\"\n \"Gitk - a commit viewer for git\\n\"\n \"\\n\"\n-\"Copyright ©9 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Use and redistribute under the terms of the GNU General Public License\"\n msgstr \"\"\n \"\\n\"\n \"Gitk - en incheckningsvisare för git\\n\"\n \"\\n\"\n-\"Copyright © 2005-2009 Paul Mackerras\\n\"\n+\"Copyright \\\\u00a9 2005-2010 Paul Mackerras\\n\"\n \"\\n\"\n \"Använd och vidareförmedla enligt villkoren i GNU General Public License\"\n \n-- \n1.6.6\n"},{"id":"137312","messageId":"20100320061452.GA26224@progeny.tock","threadId":"23000","inReplyTo":"1268418709-4998-3-git-send-email-patthoyts@users.sourceforge.net","subject":"[PATCH] gitk: Fix display of copyright symbol in non-utf-8 locales","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-20T06:14:52Z","receivedAt":"2010-03-20T06:14:52Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Unless the ‘-encoding’ option is supplied, wish interprets source\nfiles using the default system encoding, which depends on locale.\nThis can cause the copyright symbol to display incorrectly in gitk’s\nabout dialog.\n\nThe -encoding option was added to tcl in version 8.1.\n\nReported-by: Pat Thoyts \nSigned-off-by: Jonathan Nieder \n---\nHi,\n\nPat Thoyts wrote:\n\n> The script file uses utf-8 encoding but when sourced it will be read\n> using the default system encoding which is never utf8 on windows. This\n> causes the copyright symbol to display incorrectly in the about dialog.\n\nI can reproduce this in Linux with LANG=de_DE@euro ./gitk.\n\n> Using the unicode escape sequence avoids incorrect decoding but does\n> require a double escape in the .po files.\n\nWhen I tried applying your patch, running “make update-po” seemed to\nundo its effect on translations. I am also worried about what would\nhappen in locales where ASCII is not a subset of the native character\nset, or other uses of non-ASCII characters in the translated messages.\n\nMaybe the following would be safer? I haven’t looked at the effect on\ntranslations yet.\n\n gitk | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/gitk b/gitk\nindex 1f36a3e..a7c9a9a 100755\n--- a/gitk\n+++ b/gitk\n@@ -1,6 +1,6 @@\n #!/bin/sh\n # Tcl ignores the next line -*- tcl -*- \\\n-exec wish \"$0\" -- \"$@\"\n+exec wish -encoding utf-8 \"$0\" -- \"$@\"\n \n # Copyright © 2005-2009 Paul Mackerras. All rights reserved.\n # This program is free software; it may be used, copied, modified\n-- \n1.7.0.2\n"},{"id":"137313","messageId":"20100320062427.GA26434@progeny.tock","threadId":"23000","inReplyTo":"20100320061452.GA26224@progeny.tock","subject":"[PATCH squash] gitk: Update makefile to keep -encoding option for gitk-wish","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-20T06:24:58Z","receivedAt":"2010-03-20T06:24:58Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Jonathan Nieder wrote:\n\n> I can reproduce this in Linux with LANG=de_DE@euro ./gitk.\n\nThis should be squashed in to fix LANG=de_DE@euro ./gitk-wish, too.\nSorry for the trouble.\n\nSigned-off-by: Jonathan Nieder \n\n Makefile | 2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Makefile b/Makefile\nindex e1b6045..1cc922c 100644\n--- a/Makefile\n+++ b/Makefile\n@@ -53,7 +53,7 @@ clean::\n \n gitk-wish: gitk\n \t$(QUIET_GEN)$(RM) $@ $@+ && \\\n-\tsed -e '1,3s|^exec .* \"$$0\"|exec $(subst |,'\\|',$(TCLTK_PATH_SQ)) \"$$0\"|' $@+ && \\\n+\tsed -e '1,3s|^exec .* -encoding|exec $(subst |,'\\|',$(TCLTK_PATH_SQ)) -encoding|' $@+ && \\\n \tchmod +x $@+ && \\\n \tmv -f $@+ $@\n \n-- \n1.7.0.2\n"},{"id":"137314","messageId":"20100320063256.GA26519@progeny.tock","threadId":"23000","inReplyTo":"1268418709-4998-1-git-send-email-patthoyts@users.sourceforge.net","subject":"Re: [PATCH 1/3] gitk: Avoid calling tk_setPalette on Windows","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-20T06:32:56Z","receivedAt":"2010-03-20T06:32:56Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Pat Thoyts wrote:\n\n> Signed-off-by: Pat Thoyts \n> ---\n> gitk | 1 +\n> 1 files changed, 1 insertions(+), 0 deletions(-)\n> \n> diff --git a/gitk b/gitk\n> index 1f36a3e..dc9b8a8 100755\n> --- a/gitk\n> +++ b/gitk\n> @@ -10845,10 +10845,11 @@ proc setselbg {c} {\n> \n> # This sets the background color and the color scheme for the whole UI.\n> # For some reason, tk_setPalette chooses a nasty dark red for selectColor\n> # if we don't specify one ourselves, which makes the checkbuttons and\n> # radiobuttons look bad. This chooses white for selectColor if the\n> # background color is light, or black if it is dark.\n> proc setui {c} {\n> + if {[tk windowingsystem] eq \"win32\"} { return }\n\nDo you know whether the situation described by the comment has been addressed\nfor Tk? Maybe gitk could check for an appropriate Tk version to avoid this\nworkaround on all platforms.\n"},{"id":"137592","messageId":"87tys8c6ki.fsf@fox.patthoyts.tk","threadId":"23000","inReplyTo":"20100320063256.GA26519@progeny.tock","subject":"Re: [PATCH 1/3] gitk: Avoid calling tk_setPalette on Windows","fromName":"Pat Thoyts","fromEmail":"patthoyts@users.sourceforge.net","sentAt":"2010-03-22T22:52:45Z","receivedAt":"2010-03-22T22:52:45Z","isPatch":true,"sender":{"key":"patthoyts@users.sourceforge.net","avatar":"https://avatars.githubusercontent.com/u/30739?v=4"},"body":"Jonathan Nieder writes:\n\n>Pat Thoyts wrote:\n>\n>> Signed-off-by: Pat Thoyts \n>> ---\n>> gitk | 1 +\n>> 1 files changed, 1 insertions(+), 0 deletions(-)\n>> \n>> diff --git a/gitk b/gitk\n>> index 1f36a3e..dc9b8a8 100755\n>> --- a/gitk\n>> +++ b/gitk\n>> @@ -10845,10 +10845,11 @@ proc setselbg {c} {\n>> \n>> # This sets the background color and the color scheme for the whole UI.\n>> # For some reason, tk_setPalette chooses a nasty dark red for selectColor\n>> # if we don't specify one ourselves, which makes the checkbuttons and\n>> # radiobuttons look bad. This chooses white for selectColor if the\n>> # background color is light, or black if it is dark.\n>> proc setui {c} {\n>> + if {[tk windowingsystem] eq \"win32\"} { return }\n>\n>Do you know whether the situation described by the comment has been addressed\n>for Tk? Maybe gitk could check for an appropriate Tk version to avoid this\n>workaround on all platforms.\n\nThis has never been true for Windows which has always used the native\nimages for check and radio buttons. So the comment there is relevant\nonly to X11. I believe in Tk 8.5 the check/radio buttons now use\nimages by default (the ttk versions will look appropriate to the\ncurrent theme - however that may be defined).\n\nHowever, messing about with this on X11 would need testing that I\ncan't do on Windows. Hence the suggested fix.\n\n-- \nPat Thoyts http://www.patthoyts.tk/\nPGP fingerprint 2C 6E 98 07 2C 59 C8 97 10 CE 11 E6 04 E0 B9 DD\n"},{"id":"137593","messageId":"87pr2wc6bo.fsf@fox.patthoyts.tk","threadId":"23000","inReplyTo":"20100320061452.GA26224@progeny.tock","subject":"Re: [PATCH] gitk: Fix display of copyright symbol in non-utf-8 locales","fromName":"Pat Thoyts","fromEmail":"patthoyts@users.sourceforge.net","sentAt":"2010-03-22T22:58:03Z","receivedAt":"2010-03-22T22:58:03Z","isPatch":true,"sender":{"key":"patthoyts@users.sourceforge.net","avatar":"https://avatars.githubusercontent.com/u/30739?v=4"},"body":"Jonathan Nieder writes:\n\n>Unless the ‘-encoding’ option is supplied, wish interprets source\n>files using the default system encoding, which depends on locale.\n>This can cause the copyright symbol to display incorrectly in gitk’s\n>about dialog.\n>\n>The -encoding option was added to tcl in version 8.1.\n>\n>Reported-by: Pat Thoyts \n>Signed-off-by: Jonathan Nieder \n>---\n>Hi,\n>\n>Pat Thoyts wrote:\n>\n>> The script file uses utf-8 encoding but when sourced it will be read\n>> using the default system encoding which is never utf8 on windows. This\n>> causes the copyright symbol to display incorrectly in the about dialog.\n>\n>I can reproduce this in Linux with LANG=de_DE@euro ./gitk.\n>\n>> Using the unicode escape sequence avoids incorrect decoding but does\n>> require a double escape in the .po files.\n>\n>When I tried applying your patch, running “make update-po” seemed to\n>undo its effect on translations. I am also worried about what would\n>happen in locales where ASCII is not a subset of the native character\n>set, or other uses of non-ASCII characters in the translated messages.\n>\n>Maybe the following would be safer? I haven’t looked at the effect on\n>translations yet.\n>\n> gitk | 2 +-\n> 1 files changed, 1 insertions(+), 1 deletions(-)\n>\n>diff --git a/gitk b/gitk\n>index 1f36a3e..a7c9a9a 100755\n>--- a/gitk\n>+++ b/gitk\n>@@ -1,6 +1,6 @@\n> #!/bin/sh\n> # Tcl ignores the next line -*- tcl -*- \\\n>-exec wish \"$0\" -- \"$@\"\n>+exec wish -encoding utf-8 \"$0\" -- \"$@\"\n> \n> # Copyright © 2005-2009 Paul Mackerras. All rights reserved.\n> # This program is free software; it may be used, copied, modified\n>-- \n>1.7.0.2\n\nI would much rather see any fault in the makefile fixed than use\nsource -encoding in all the various places this script might get\nloaded from. THere are two characters in this file that are not ASCII\nand one of them is in a comment. We could even just use (c) to\nsimplify this. However, \\u00a9 is the correct unicode escape sequence\nfor tcl to get a copyright character into the script in a way that is\nindependent of the encoding. The escape sequence should be passed\nthrough unchanged in any translations - tcl wants to see exactly\n\\u00a9 and everyone is using a copyright glyph here. \n-- \nPat Thoyts http://www.patthoyts.tk/\nPGP fingerprint 2C 6E 98 07 2C 59 C8 97 10 CE 11 E6 04 E0 B9 DD\n"},{"id":"137596","messageId":"20100322234517.GB2168@progeny.tock","threadId":"23000","inReplyTo":"87pr2wc6bo.fsf@fox.patthoyts.tk","subject":"Re: [PATCH] gitk: Fix display of copyright symbol in non-utf-8 locales","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-22T23:45:17Z","receivedAt":"2010-03-22T23:45:17Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Pat Thoyts wrote:\n> Jonathan Nieder writes:\n\n>> --- a/gitk\n>> +++ b/gitk\n>> @@ -1,6 +1,6 @@\n>> #!/bin/sh\n>> # Tcl ignores the next line -*- tcl -*- \\\n>> -exec wish \"$0\" -- \"$@\"\n>> +exec wish -encoding utf-8 \"$0\" -- \"$@\"\n[...]\n> I would much rather see any fault in the makefile fixed than use\n> source -encoding in all the various places this script might get\n> loaded from.\n\nGood point: git-gui.sh does not use ‘source’ but runs the moral\nequivalent of\n \n set wish [info executable]\n eval exec $wish gitk $revs \"--\" \"--\"\n\nto avoid starting a shell. Is it really so expensive to start a\nshell? This means that, for example, if some insane person were to\nrewrite gitk in C, then git gui would have to be adjusted before the\nresult could be used.\n\n> THere are two characters in this file that are not ASCII\n> and one of them is in a comment. We could even just use (c) to\n> simplify this. However, \\u00a9 is the correct unicode escape sequence\n> for tcl to get a copyright character into the script in a way that is\n> independent of the encoding. The escape sequence should be passed\n> through unchanged in any translations - tcl wants to see exactly\n> \\u00a9 and everyone is using a copyright glyph here. \n\nSo we should fix the Makefile. The problem is that everything\nTcl-related seems to assume files it sees will be in either the\ncurrent locale’s encoding or ISO-8859-1 unless told otherwise, as far\nas I can tell. The entire gitk source tree, on the other hand, is in\nUTF-8.\n\nIdeas?\nJonathan\n"},{"id":"137603","messageId":"20100323022903.GA5213@progeny.tock","threadId":"23000","inReplyTo":"87tys8c6ki.fsf@fox.patthoyts.tk","subject":"Re: [PATCH 1/3] gitk: Avoid calling tk_setPalette on Windows","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-03-23T02:29:03Z","receivedAt":"2010-03-23T02:29:03Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Pat Thoyts wrote:\n\n>>> # For some reason, tk_setPalette chooses a nasty dark red for selectColor\n>>> # if we don't specify one ourselves, which makes the checkbuttons and\n>>> # radiobuttons look bad. This chooses white for selectColor if the\n>>> # background color is light, or black if it is dark.\n[...]\n> This has never been true for Windows which has always used the native\n> images for check and radio buttons. So the comment there is relevant\n> only to X11. I believe in Tk 8.5 the check/radio buttons now use\n> images by default (the ttk versions will look appropriate to the\n> current theme - however that may be defined).\n> \n> However, messing about with this on X11 would need testing that I\n> can't do on Windows. Hence the suggested fix.\n\nThanks for the pointers. Sp it seems that the “interface color” setting\ndoes not have the desired effect when themed widgets are enabled. You\ncan see some screenshots here:\n\n http://repo.or.cz/w/git/jrn.git/commit/gitk-snapshots\n\nThe text fields, buttons, and drop-down boxes retain the traditional\ngrey background, while the menu bar, “SHA1 ID” label, and “Lines of\ncontext” selector do change color. The effect is very strange.\n\nWhen themed widgets are disabled, the tk_setPalette issue described in\nthe comment still applies.\n\nSo I can understand better: are you saying in Windows themed widgets\nare always used?\n\nJonathan\n"}]}