{"thread":{"id":"42196","subject":"gitk: \"lime\" color incompatible with older Tk versions","startedAt":"2016-05-01T17:03:50Z","lastAt":"2017-01-18T18:41:53Z","messageCount":11,"participants":["Andrew Janke","Stefan Beller","David Aguilar","Junio C Hamano","Paul Mackerras"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"285121","messageId":"03babaa1-9011-0010-c4b3-6cad8109d3ab@apjanke.net","threadId":"42196","inReplyTo":null,"subject":"gitk: \"lime\" color incompatible with older Tk versions","fromName":"Andrew Janke","fromEmail":"floss@apjanke.net","sentAt":"2016-05-01T17:03:50Z","receivedAt":"2016-05-01T17:03:50Z","isPatch":false,"sender":{"key":"floss@apjanke.net","avatar":"https://gravatar.com/avatar/cce96357f6822eb4d08552395bda5a987bf4aa3e98eb42ad062ea8cf352c514b?d=mp&s=160"},"body":"Hi, git folks,\n\nI'm having trouble running gitk on Mac OS X 10.9.5. The gitk program \nuses the color \"lime\", which is not present in older versions of Tk, \napparently including the Tk 8.5 which ships with 10.9.\n\nWhen I try to launch it, I get this error.\n\n|$ gitk Error in startup script: unknown color name \"lime\" (processing \n\"-fore\" option) invoked from within \"$ctext tag conf m2 -fore [lindex \n$mergecolors 2]\" (procedure \"makewindow\" line 347) invoked from within \n\"makewindow\" (file \"/usr/local/bin/gitk\" line 12434)|\n\nThis compatibility problem was noted before back in 2012, in \nhttp://www.mail-archive.com/git%40vger.kernel.org/msg14496.html.\n\nWould you consider switching from lime to a hex value color, for \ncompatibility with users of older versions of Tk? A patch to do so is \nbelow; only the file gitk-git/gitk needs to be changed.\n\nCheers,\nAndrew Janke\n\n\ndiff --git a/gitk-git/gitk b/gitk-git/gitk\nindex 805a1c7..db5560d 100755\n--- a/gitk-git/gitk\n+++ b/gitk-git/gitk\n@@ -2265,7 +2265,7 @@ proc makewindow {} {\n      set h [expr {[font metrics uifont -linespace] + 2}]\n      set progresscanv .tf.bar.progress\n      canvas $progresscanv -relief sunken -height $h -borderwidth 2\n-    set progressitem [$progresscanv create rect -1 0 0 $h -fill lime]\n+    set progressitem [$progresscanv create rect -1 0 0 $h -fill \"#99FF00\"]\n      set fprogitem [$progresscanv create rect -1 0 0 $h -fill yellow]\n      set rprogitem [$progresscanv create rect -1 0 0 $h -fill red]\n      }\n@@ -3397,7 +3397,7 @@ set rectmask {\n         0x00, 0x00, 0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f,\n         0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f, 0x00, 0x00};\n  }\n-image create bitmap reficon-H -background black -foreground lime \\\n+image create bitmap reficon-H -background black -foreground \"#99FF00\" \\\n      -data $rectdata -maskdata $rectmask\n  image create bitmap reficon-o -background black -foreground \"#ddddff\" \\\n      -data $rectdata -maskdata $rectmask\n@@ -12188,7 +12188,7 @@ if {[tk windowingsystem] eq \"aqua\"} {\n      set extdifftool \"meld\"\n  }\n\n-set colors {lime red blue magenta darkgrey brown orange}\n+set colors {\"#99FF00\" red blue magenta darkgrey brown orange}\n  if {[tk windowingsystem] eq \"win32\"} {\n      set uicolor SystemButtonFace\n      set uifgcolor SystemButtonText\n@@ -12206,12 +12206,12 @@ if {[tk windowingsystem] eq \"win32\"} {\n  }\n  set diffcolors {red \"#00a000\" blue}\n  set diffcontext 3\n-set mergecolors {red blue lime purple brown \"#009090\" magenta \"#808000\" \n\"#009000\" \"#ff0080\" cyan \"#b07070\" \"#70b0f0\" \"#70f0b0\" \"#f0b070\" \"#ff70b0\"}\n+set mergecolors {red blue \"#99FF00\" purple brown \"#009090\" magenta \n\"#808000\" \"#009000\" \"#ff0080\" cyan \"#b07070\" \"#70b0f0\" \"#70f0b0\" \n\"#f0b070\" \"#ff70b0\"}\n  set ignorespace 0\n  set worddiff \"\"\n  set markbgcolor \"#e0e0ff\"\n\n-set headbgcolor lime\n+set headbgcolor \"#99FF00\"\n  set headfgcolor black\n  set headoutlinecolor black\n  set remotebgcolor #ffddaa\n@@ -12226,7 +12226,7 @@ set linehoverfgcolor black\n  set linehoveroutlinecolor black\n  set mainheadcirclecolor yellow\n  set workingfilescirclecolor red\n-set indexcirclecolor lime\n+set indexcirclecolor \"#99FF00\"\n  set circlecolors {white blue gray blue blue}\n  set linkfgcolor blue\n  set circleoutlinecolor $fgcolor\n"},{"id":"285161","messageId":"CAGZ79kaO9T+Qc=M6s_ZdpAfLZCVQEYNF=zNxDWArDmsA7jjCWg@mail.gmail.com","threadId":"42196","inReplyTo":"03babaa1-9011-0010-c4b3-6cad8109d3ab@apjanke.net","subject":"Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"Stefan Beller","fromEmail":"sbeller@google.com","sentAt":"2016-05-02T16:20:43Z","receivedAt":"2016-05-02T16:20:43Z","isPatch":false,"sender":{"key":"stefanbeller@gmail.com","avatar":"https://avatars.githubusercontent.com/u/455868?v=4"},"body":"+ Paul Mackerras, who maintains gitk\n\nOn Sun, May 1, 2016 at 10:03 AM, Andrew Janke <floss@apjanke.net> wrote:\n> Hi, git folks,\n>\n> I'm having trouble running gitk on Mac OS X 10.9.5. The gitk program uses\n> the color \"lime\", which is not present in older versions of Tk, apparently\n> including the Tk 8.5 which ships with 10.9.\n>\n> When I try to launch it, I get this error.\n>\n> |$ gitk Error in startup script: unknown color name \"lime\" (processing\n> \"-fore\" option) invoked from within \"$ctext tag conf m2 -fore [lindex\n> $mergecolors 2]\" (procedure \"makewindow\" line 347) invoked from within\n> \"makewindow\" (file \"/usr/local/bin/gitk\" line 12434)|\n>\n> This compatibility problem was noted before back in 2012, in\n> http://www.mail-archive.com/git%40vger.kernel.org/msg14496.html.\n>\n> Would you consider switching from lime to a hex value color, for\n> compatibility with users of older versions of Tk? A patch to do so is below;\n> only the file gitk-git/gitk needs to be changed.\n>\n> Cheers,\n> Andrew Janke\n>\n>\n> diff --git a/gitk-git/gitk b/gitk-git/gitk\n> index 805a1c7..db5560d 100755\n> --- a/gitk-git/gitk\n> +++ b/gitk-git/gitk\n> @@ -2265,7 +2265,7 @@ proc makewindow {} {\n>      set h [expr {[font metrics uifont -linespace] + 2}]\n>      set progresscanv .tf.bar.progress\n>      canvas $progresscanv -relief sunken -height $h -borderwidth 2\n> -    set progressitem [$progresscanv create rect -1 0 0 $h -fill lime]\n> +    set progressitem [$progresscanv create rect -1 0 0 $h -fill \"#99FF00\"]\n>      set fprogitem [$progresscanv create rect -1 0 0 $h -fill yellow]\n>      set rprogitem [$progresscanv create rect -1 0 0 $h -fill red]\n>      }\n> @@ -3397,7 +3397,7 @@ set rectmask {\n>         0x00, 0x00, 0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f,\n>         0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f, 0xfc, 0x0f, 0x00, 0x00};\n>  }\n> -image create bitmap reficon-H -background black -foreground lime \\\n> +image create bitmap reficon-H -background black -foreground \"#99FF00\" \\\n>      -data $rectdata -maskdata $rectmask\n>  image create bitmap reficon-o -background black -foreground \"#ddddff\" \\\n>      -data $rectdata -maskdata $rectmask\n> @@ -12188,7 +12188,7 @@ if {[tk windowingsystem] eq \"aqua\"} {\n>      set extdifftool \"meld\"\n>  }\n>\n> -set colors {lime red blue magenta darkgrey brown orange}\n> +set colors {\"#99FF00\" red blue magenta darkgrey brown orange}\n>  if {[tk windowingsystem] eq \"win32\"} {\n>      set uicolor SystemButtonFace\n>      set uifgcolor SystemButtonText\n> @@ -12206,12 +12206,12 @@ if {[tk windowingsystem] eq \"win32\"} {\n>  }\n>  set diffcolors {red \"#00a000\" blue}\n>  set diffcontext 3\n> -set mergecolors {red blue lime purple brown \"#009090\" magenta \"#808000\"\n> \"#009000\" \"#ff0080\" cyan \"#b07070\" \"#70b0f0\" \"#70f0b0\" \"#f0b070\" \"#ff70b0\"}\n> +set mergecolors {red blue \"#99FF00\" purple brown \"#009090\" magenta\n> \"#808000\" \"#009000\" \"#ff0080\" cyan \"#b07070\" \"#70b0f0\" \"#70f0b0\" \"#f0b070\"\n> \"#ff70b0\"}\n>  set ignorespace 0\n>  set worddiff \"\"\n>  set markbgcolor \"#e0e0ff\"\n>\n> -set headbgcolor lime\n> +set headbgcolor \"#99FF00\"\n>  set headfgcolor black\n>  set headoutlinecolor black\n>  set remotebgcolor #ffddaa\n> @@ -12226,7 +12226,7 @@ set linehoverfgcolor black\n>  set linehoveroutlinecolor black\n>  set mainheadcirclecolor yellow\n>  set workingfilescirclecolor red\n> -set indexcirclecolor lime\n> +set indexcirclecolor \"#99FF00\"\n>  set circlecolors {white blue gray blue blue}\n>  set linkfgcolor blue\n>  set circleoutlinecolor $fgcolor\n>\n> --\n> To unsubscribe from this list: send the line \"unsubscribe git\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at  http://vger.kernel.org/majordomo-info.html\n"},{"id":"309351","messageId":"20170113112043.j7nowdilolswyk2k@gmail.com","threadId":"42196","inReplyTo":"CAGZ79kaO9T+Qc=M6s_ZdpAfLZCVQEYNF=zNxDWArDmsA7jjCWg@mail.gmail.com","subject":"Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2017-01-13T11:20:43Z","receivedAt":"2017-01-13T11:30:02Z","isPatch":false,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"On Mon, May 02, 2016 at 09:20:43AM -0700, Stefan Beller wrote:\n> + Paul Mackerras, who maintains gitk\n> \n> On Sun, May 1, 2016 at 10:03 AM, Andrew Janke <floss@apjanke.net> wrote:\n> > Hi, git folks,\n> >\n> > I'm having trouble running gitk on Mac OS X 10.9.5. The gitk program uses\n> > the color \"lime\", which is not present in older versions of Tk, apparently\n> > including the Tk 8.5 which ships with 10.9.\n\nPing.. it would be nice to get this patch applied.\nI can verify that gitk on Mac OS X 10.11 also has this problem.\ngitk is usually pretty good about backwards-compatibility.\n\n> > This compatibility problem was noted before back in 2012, in\n> > http://www.mail-archive.com/git%40vger.kernel.org/msg14496.html.\n> >\n> > Would you consider switching from lime to a hex value color, for\n> > compatibility with users of older versions of Tk? A patch to do so is below;\n> > only the file gitk-git/gitk needs to be changed.\n\nI can recreate and resend this patch if needed; it's simply:\n:%s/lime/\"#99FF00\"/g\n\nWould a re-roll of this patch be accepted, or is it not worth\nbothering?\n\nGoogle for \"gitk lime\" to get a taste for some of the fallout\ncaused by this problem.\n\nThe fact that multiple pages, with different OS's, have examples\nof users stumbling over this change is a good hint that it's\nworth fixing.\n\nThoughts?\n-- \nDavid\n"},{"id":"309411","messageId":"20170114084825.lcecrbtxny3ntulf@gmail.com","threadId":"42196","inReplyTo":"20170113112043.j7nowdilolswyk2k@gmail.com","subject":"gitk pull request // was: Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2017-01-14T08:48:25Z","receivedAt":"2017-01-14T08:48:38Z","isPatch":false,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"On Fri, Jan 13, 2017 at 03:20:43AM -0800, David Aguilar wrote:\n> \n> Ping.. it would be nice to get this patch applied.\n\nSorry for the noise, and thank you Paul for the fix.\nThis was already fixed by Paul in gitk@22a713c72df.\n\nI'm sure Junio will merge gitk.git into git.git soon enough so I\ncan sit tight until then, but while I'm here I might as well\nsend out a pull request:\n\nThe following changes since commit 22a713c72df8b6799c59287c50cee44c4a6db51e:\n\n  gitk: Follow themed bgcolor in help dialogs (2016-03-19 14:12:21 +1100)\n\nare available in the git repository at:\n\n  git://ozlabs.org/~paulus/gitk.git \n\nfor you to fetch changes up to fbf426478e540f4737860dae622603cc0daba3d2:\n\n  gitk: Update copyright notice to 2016 (2016-12-12 20:46:42 +1100)\n\n----------------------------------------------------------------\nMarkus Hitter (3):\n      gitk: Turn off undo manager in the text widget\n      gitk: Remove closed file descriptors from $blobdifffd\n      gitk: Clear array 'commitinfo' on reload\n\nPaul Mackerras (2):\n      gitk: Use explicit RGB green instead of \"lime\"\n      gitk: Update copyright notice to 2016\n\nRogier Goossens (3):\n      gitk: Add a 'rename' option to the branch context menu\n      gitk: Allow checking out a remote branch\n      gitk: Include commit title in branch dialog\n\nSatoshi Yasushima (1):\n      gitk: Fix Japanese translation for \"marked commit\"\n\nStefan Dotterweich (1):\n      gitk: Fix missing commits when using -S or -G\n\nVasco Almeida (2):\n      gitk: Makefile: create install bin directory\n      gitk: Add Portuguese translation\n\n Makefile    |    1 +\n gitk        |  166 +++++--\n po/bg.po    |    4 +-\n po/ca.po    |    6 +-\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    |   13 +-\n po/pt_br.po |    4 +-\n po/pt_pt.po | 1376 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++\n po/ru.po    |    4 +-\n po/sv.po    |    8 +-\n po/vi.po    |    4 +-\n 15 files changed, 1549 insertions(+), 57 deletions(-)\n create mode 100644 po/pt_pt.po\n\nThanks,\n-- \nDavid\n"},{"id":"309428","messageId":"xmqq37gldp8g.fsf@gitster.mtv.corp.google.com","threadId":"42196","inReplyTo":"20170114084825.lcecrbtxny3ntulf@gmail.com","subject":"Re: gitk pull request // was: Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-01-15T02:35:43Z","receivedAt":"2017-01-15T02:35:55Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"David Aguilar <davvid@gmail.com> writes:\n\n> On Fri, Jan 13, 2017 at 03:20:43AM -0800, David Aguilar wrote:\n>> \n>> Ping.. it would be nice to get this patch applied.\n>\n> Sorry for the noise, and thank you Paul for the fix.\n> This was already fixed by Paul in gitk@22a713c72df.\n>\n> I'm sure Junio will merge gitk.git into git.git soon enough so I\n> can sit tight until then, but while I'm here I might as well\n> send out a pull request:\n>\n> The following changes since commit 22a713c72df8b6799c59287c50cee44c4a6db51e:\n>\n>   gitk: Follow themed bgcolor in help dialogs (2016-03-19 14:12:21 +1100)\n>\n> are available in the git repository at:\n>\n>   git://ozlabs.org/~paulus/gitk.git \n>\n> for you to fetch changes up to fbf426478e540f4737860dae622603cc0daba3d2:\n>\n>   gitk: Update copyright notice to 2016 (2016-12-12 20:46:42 +1100)\n\nPinging Paul to signal me that his tree is ready to pull from is\nappreciated, and asking Paul if his tree is ready to be pulled and\nthen relaying his answer to me is also fine, but I am sensing that\nthis message is neither.  So let me double check.\n\nPaul, is it a good time to pull, or do you still have something not\npublished yet that should go together with what you have already\nqueued?\n\nThanks.\n\n>\n> ----------------------------------------------------------------\n> Markus Hitter (3):\n>       gitk: Turn off undo manager in the text widget\n>       gitk: Remove closed file descriptors from $blobdifffd\n>       gitk: Clear array 'commitinfo' on reload\n>\n> Paul Mackerras (2):\n>       gitk: Use explicit RGB green instead of \"lime\"\n>       gitk: Update copyright notice to 2016\n>\n> Rogier Goossens (3):\n>       gitk: Add a 'rename' option to the branch context menu\n>       gitk: Allow checking out a remote branch\n>       gitk: Include commit title in branch dialog\n>\n> Satoshi Yasushima (1):\n>       gitk: Fix Japanese translation for \"marked commit\"\n>\n> Stefan Dotterweich (1):\n>       gitk: Fix missing commits when using -S or -G\n>\n> Vasco Almeida (2):\n>       gitk: Makefile: create install bin directory\n>       gitk: Add Portuguese translation\n>\n>  Makefile    |    1 +\n>  gitk        |  166 +++++--\n>  po/bg.po    |    4 +-\n>  po/ca.po    |    6 +-\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    |   13 +-\n>  po/pt_br.po |    4 +-\n>  po/pt_pt.po | 1376 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++\n>  po/ru.po    |    4 +-\n>  po/sv.po    |    8 +-\n>  po/vi.po    |    4 +-\n>  15 files changed, 1549 insertions(+), 57 deletions(-)\n>  create mode 100644 po/pt_pt.po\n>\n> Thanks,\n"},{"id":"309465","messageId":"20170116031706.GA3322@fergus.ozlabs.ibm.com","threadId":"42196","inReplyTo":"xmqq37gldp8g.fsf@gitster.mtv.corp.google.com","subject":"Re: gitk pull request // was: Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"Paul Mackerras","fromEmail":"paulus@ozlabs.org","sentAt":"2017-01-16T03:17:06Z","receivedAt":"2017-01-16T03:17:28Z","isPatch":false,"sender":{"key":"paulus@ozlabs.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"Hi Junio,\n\nOn Sat, Jan 14, 2017 at 06:35:43PM -0800, Junio C Hamano wrote:\n> David Aguilar <davvid@gmail.com> writes:\n> \n> > On Fri, Jan 13, 2017 at 03:20:43AM -0800, David Aguilar wrote:\n> >> \n> >> Ping.. it would be nice to get this patch applied.\n> >\n> > Sorry for the noise, and thank you Paul for the fix.\n> > This was already fixed by Paul in gitk@22a713c72df.\n> >\n> > I'm sure Junio will merge gitk.git into git.git soon enough so I\n> > can sit tight until then, but while I'm here I might as well\n> > send out a pull request:\n> >\n> > The following changes since commit 22a713c72df8b6799c59287c50cee44c4a6db51e:\n> >\n> >   gitk: Follow themed bgcolor in help dialogs (2016-03-19 14:12:21 +1100)\n> >\n> > are available in the git repository at:\n> >\n> >   git://ozlabs.org/~paulus/gitk.git \n> >\n> > for you to fetch changes up to fbf426478e540f4737860dae622603cc0daba3d2:\n> >\n> >   gitk: Update copyright notice to 2016 (2016-12-12 20:46:42 +1100)\n> \n> Pinging Paul to signal me that his tree is ready to pull from is\n> appreciated, and asking Paul if his tree is ready to be pulled and\n> then relaying his answer to me is also fine, but I am sensing that\n> this message is neither.  So let me double check.\n> \n> Paul, is it a good time to pull, or do you still have something not\n> published yet that should go together with what you have already\n> queued?\n\nI recently pushed out one more commit to update the Russian\ntranslation from Dimitriy Ryazantcev.  The head is now 8fef3f36b779.\nI have a couple more series that I am currently reviewing, but nothing\nimmediately ready to publish.  It would be a good time for you to do a\npull, since the \"lime\" color fix and the memory consumption fixes\nshould be helpful for a lot of people.\n\nThanks,\nPaul.\n"},{"id":"309473","messageId":"xmqqy3yba1jg.fsf@gitster.mtv.corp.google.com","threadId":"42196","inReplyTo":"20170116031706.GA3322@fergus.ozlabs.ibm.com","subject":"Re: gitk pull request // was: Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-01-16T07:48:03Z","receivedAt":"2017-01-16T07:48:16Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Paul Mackerras <paulus@ozlabs.org> writes:\n\n>> Paul, is it a good time to pull, or do you still have something not\n>> published yet that should go together with what you have already\n>> queued?\n>\n> I recently pushed out one more commit to update the Russian\n> translation from Dimitriy Ryazantcev.  The head is now 8fef3f36b779.\n> I have a couple more series that I am currently reviewing, but nothing\n> immediately ready to publish.  It would be a good time for you to do a\n> pull, since the \"lime\" color fix and the memory consumption fixes\n> should be helpful for a lot of people.\n\nThanks.  I did want to get the memory consumption fix sooner rather\nthan later, and this is very much appreciated.\n\nPulled.\n"},{"id":"309533","messageId":"xmqq4m0xpmbz.fsf@gitster.mtv.corp.google.com","threadId":"42196","inReplyTo":"xmqqy3yba1jg.fsf@gitster.mtv.corp.google.com","subject":"Re: gitk pull request // was: Re: gitk: \"lime\" color incompatible with older Tk versions","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-01-17T18:34:24Z","receivedAt":"2017-01-17T18:34:34Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Paul Mackerras <paulus@ozlabs.org> writes:\n>\n>>> Paul, is it a good time to pull, or do you still have something not\n>>> published yet that should go together with what you have already\n>>> queued?\n>>\n>> I recently pushed out one more commit to update the Russian\n>> translation from Dimitriy Ryazantcev.  The head is now 8fef3f36b779.\n>> I have a couple more series that I am currently reviewing, but nothing\n>> immediately ready to publish.  It would be a good time for you to do a\n>> pull, since the \"lime\" color fix and the memory consumption fixes\n>> should be helpful for a lot of people.\n>\n> Thanks.  I did want to get the memory consumption fix sooner rather\n> than later, and this is very much appreciated.\n>\n> Pulled.\n\nHmph.  I am getting these:\n\n        SUBDIR gitk-git\n    Generating catalog po/sv.msg\n    msgfmt --statistics --tcl po/sv.po -l sv -d po/\n    po/sv.po:1388: duplicate message definition...\n    po/sv.po:380: ...this is the location of the first definition\n    msgfmt: found 1 fatal error\n    make[1]: *** [po/sv.msg] Error 1\n    make: *** [all] Error 2\n\nAnybody else see this?\n"},{"id":"309610","messageId":"20170118035245.1757-1-davvid@gmail.com","threadId":"42196","inReplyTo":"xmqq4m0xpmbz.fsf@gitster.mtv.corp.google.com","subject":"[PATCH] gitk: remove translated message from comments","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2017-01-18T03:52:45Z","receivedAt":"2017-01-18T03:53:09Z","isPatch":true,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"\"make update-po\" fails because a previously untranslated string\nhas now been translated:\n\n\tUpdating po/sv.po\n\tpo/sv.po:1388: duplicate message definition...\n\tpo/sv.po:380: ...this is the location of the first definition\n\nRemove the duplicate message definition.\n\nReported-by: Junio C Hamano <gitster@pobox.com>\nSigned-off-by: David Aguilar <davvid@gmail.com>\n---\n po/sv.po | 15 ---------------\n 1 file changed, 15 deletions(-)\n\ndiff --git a/po/sv.po b/po/sv.po\nindex 32fc752..2a06fe5 100644\n--- a/po/sv.po\n+++ b/po/sv.po\n@@ -1385,21 +1385,6 @@ msgstr \"Felaktiga argument till gitk:\"\n #~ msgid \"mc\"\n #~ msgstr \"mc\"\n \n-#~ msgid \"\"\n-#~ \"\\n\"\n-#~ \"Gitk - a commit viewer for git\\n\"\n-#~ \"\\n\"\n-#~ \"Copyright © 2005-2016 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-2016 Paul Mackerras\\n\"\n-#~ \"\\n\"\n-#~ \"Använd och vidareförmedla enligt villkoren i GNU General Public License\"\n-\n #~ msgid \"next\"\n #~ msgstr \"nästa\"\n \n-- \n2.11.0.536.gaf746e49c2\n\n"},{"id":"309614","messageId":"20170118101515.GA12161@fergus.ozlabs.ibm.com","threadId":"42196","inReplyTo":"20170118035245.1757-1-davvid@gmail.com","subject":"Re: [PATCH] gitk: remove translated message from comments","fromName":"Paul Mackerras","fromEmail":"paulus@ozlabs.org","sentAt":"2017-01-18T10:15:15Z","receivedAt":"2017-01-18T10:15:45Z","isPatch":true,"sender":{"key":"paulus@ozlabs.org","avatar":"https://avatars.githubusercontent.com/u/1606439?v=4"},"body":"On Tue, Jan 17, 2017 at 07:52:45PM -0800, David Aguilar wrote:\n> \"make update-po\" fails because a previously untranslated string\n> has now been translated:\n> \n> \tUpdating po/sv.po\n> \tpo/sv.po:1388: duplicate message definition...\n> \tpo/sv.po:380: ...this is the location of the first definition\n> \n> Remove the duplicate message definition.\n> \n> Reported-by: Junio C Hamano <gitster@pobox.com>\n> Signed-off-by: David Aguilar <davvid@gmail.com>\n\nThanks, applied.\n\nJunio, please do a pull from my repository to get this fix.\nThe new head is 7f03c6e32891.\n\nPaul.\n"},{"id":"309637","messageId":"xmqqd1fkb4vd.fsf@gitster.mtv.corp.google.com","threadId":"42196","inReplyTo":"20170118101515.GA12161@fergus.ozlabs.ibm.com","subject":"Re: [PATCH] gitk: remove translated message from comments","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-01-18T18:27:34Z","receivedAt":"2017-01-18T18:41:53Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Paul Mackerras <paulus@ozlabs.org> writes:\n\n> On Tue, Jan 17, 2017 at 07:52:45PM -0800, David Aguilar wrote:\n>> \"make update-po\" fails because a previously untranslated string\n>> has now been translated:\n>> \n>> \tUpdating po/sv.po\n>> \tpo/sv.po:1388: duplicate message definition...\n>> \tpo/sv.po:380: ...this is the location of the first definition\n>> \n>> Remove the duplicate message definition.\n>> \n>> Reported-by: Junio C Hamano <gitster@pobox.com>\n>> Signed-off-by: David Aguilar <davvid@gmail.com>\n>\n> Thanks, applied.\n>\n> Junio, please do a pull from my repository to get this fix.\n> The new head is 7f03c6e32891.\n\nThanks.\n"}]}