* [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
@ 2015-01-22 8:19 Alex Henrie
2015-02-09 21:36 ` Alex Henrie
0 siblings, 1 reply; 7+ messages in thread
From: Alex Henrie @ 2015-01-22 8:19 UTC (permalink / raw)
To: git, patthoyts, gitster; +Cc: Alex Henrie
xgettext sees "% o" and interprets it as a placeholder for an octal
number preceded by a space. However, in this case it's not actually a
placeholder, and most translations will replace the "% o" sequence with
something else. Removing the tcl-format flag from this string prevents
tools like Poedit from freaking out when "% o" doesn't appear in the
translated string.
The corrected flag will appear in each translation's po file the next time
the translation is updated with `make update-po`.
Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>
---
gitk-git/gitk | 1 +
1 file changed, 1 insertion(+)
diff --git a/gitk-git/gitk b/gitk-git/gitk
index 78358a7..dfd458d 100755
--- a/gitk-git/gitk
+++ b/gitk-git/gitk
@@ -11237,6 +11237,7 @@ proc prefspage_general {notebook} {
${NS}::label $page.maxwidthl -text [mc "Maximum graph width (lines)"]
spinbox $page.maxwidth -from 0 -to 100 -width 4 -textvariable maxwidth
grid $page.spacer $page.maxwidthl $page.maxwidth -sticky w
+ #xgettext:no-tcl-format
${NS}::label $page.maxpctl -text [mc "Maximum graph width (% of pane)"]
spinbox $page.maxpct -from 1 -to 100 -width 4 -textvariable maxgraphpct
grid x $page.maxpctl $page.maxpct -sticky w
--
2.2.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-01-22 8:19 [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it Alex Henrie
@ 2015-02-09 21:36 ` Alex Henrie
2015-02-09 21:55 ` Junio C Hamano
0 siblings, 1 reply; 7+ messages in thread
From: Alex Henrie @ 2015-02-09 21:36 UTC (permalink / raw)
To: Git mailing list, patthoyts, Junio C Hamano
Hi,
This is just a friendly reminder that this patch has been sitting in
the mailing list archives for a couple of weeks, and it has not yet
been accepted or commented on.
-Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-02-09 21:36 ` Alex Henrie
@ 2015-02-09 21:55 ` Junio C Hamano
2015-02-16 23:27 ` Alex Henrie
2015-03-22 3:40 ` Paul Mackerras
0 siblings, 2 replies; 7+ messages in thread
From: Junio C Hamano @ 2015-02-09 21:55 UTC (permalink / raw)
To: Paul Mackerras; +Cc: Git mailing list, patthoyts, Alex Henrie
Alex Henrie <alexhenrie24@gmail.com> writes:
> This is just a friendly reminder that this patch has been sitting in
> the mailing list archives for a couple of weeks, and it has not yet
> been accepted or commented on.
I think that is because the message was not sent to the right
people, and also because the patch was made against a wrong project
;-).
I'll forward it to the gitk maintainer after digging it out of the
archive and tweaking it. Thanks.
Paul, comments?
-- >8 --
From: Alex Henrie <alexhenrie24@gmail.com>
Date: Thu, 22 Jan 2015 01:19:39 -0700
Subject: gitk: Remove tcl-format flag from a message that shouldn't have it
xgettext sees "% o" and interprets it as a placeholder for an octal
number preceded by a space. However, in this case it's not actually a
placeholder, and most translations will replace the "% o" sequence with
something else. Removing the tcl-format flag from this string prevents
tools like Poedit from freaking out when "% o" doesn't appear in the
translated string.
The corrected flag will appear in each translation's po file the next time
the translation is updated with `make update-po`.
Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>
---
gitk | 1 +
1 file changed, 1 insertion(+)
diff --git a/gitk b/gitk
index 78358a7..dfd458d 100755
--- a/gitk
+++ b/gitk
@@ -11237,6 +11237,7 @@ proc prefspage_general {notebook} {
${NS}::label $page.maxwidthl -text [mc "Maximum graph width (lines)"]
spinbox $page.maxwidth -from 0 -to 100 -width 4 -textvariable maxwidth
grid $page.spacer $page.maxwidthl $page.maxwidth -sticky w
+ #xgettext:no-tcl-format
${NS}::label $page.maxpctl -text [mc "Maximum graph width (% of pane)"]
spinbox $page.maxpct -from 1 -to 100 -width 4 -textvariable maxgraphpct
grid x $page.maxpctl $page.maxpct -sticky w
--
2.2.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-02-09 21:55 ` Junio C Hamano
@ 2015-02-16 23:27 ` Alex Henrie
2015-02-23 22:30 ` Alex Henrie
2015-03-22 3:40 ` Paul Mackerras
1 sibling, 1 reply; 7+ messages in thread
From: Alex Henrie @ 2015-02-16 23:27 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Paul Mackerras, Git mailing list, patthoyts
2015-02-09 14:55 GMT-07:00 Junio C Hamano <gitster@pobox.com>:
>
> Alex Henrie <alexhenrie24@gmail.com> writes:
>
> > This is just a friendly reminder that this patch has been sitting in
> > the mailing list archives for a couple of weeks, and it has not yet
> > been accepted or commented on.
>
> I think that is because the message was not sent to the right
> people, and also because the patch was made against a wrong project
> ;-).
>
> I'll forward it to the gitk maintainer after digging it out of the
> archive and tweaking it. Thanks.
>
> Paul, comments?
Another week and still no comments on either this patch or the gitk
Catalan translation patch. Is Paul Mackerras still actively involved
in the project?
-Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-02-16 23:27 ` Alex Henrie
@ 2015-02-23 22:30 ` Alex Henrie
2015-03-10 18:53 ` Alex Henrie
0 siblings, 1 reply; 7+ messages in thread
From: Alex Henrie @ 2015-02-23 22:30 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Paul Mackerras, Git mailing list, patthoyts
2015-02-16 16:27 GMT-07:00 Alex Henrie <alexhenrie24@gmail.com>:
> 2015-02-09 14:55 GMT-07:00 Junio C Hamano <gitster@pobox.com>:
>>
>> Alex Henrie <alexhenrie24@gmail.com> writes:
>>
>> > This is just a friendly reminder that this patch has been sitting in
>> > the mailing list archives for a couple of weeks, and it has not yet
>> > been accepted or commented on.
>>
>> I think that is because the message was not sent to the right
>> people, and also because the patch was made against a wrong project
>> ;-).
>>
>> I'll forward it to the gitk maintainer after digging it out of the
>> archive and tweaking it. Thanks.
>>
>> Paul, comments?
>
> Another week and still no comments on either this patch or the gitk
> Catalan translation patch. Is Paul Mackerras still actively involved
> in the project?
>
> -Alex
Another week and still no response. If Paul is no longer maintaining
gitk, another maintainer should be appointed.
-Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-02-23 22:30 ` Alex Henrie
@ 2015-03-10 18:53 ` Alex Henrie
0 siblings, 0 replies; 7+ messages in thread
From: Alex Henrie @ 2015-03-10 18:53 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Paul Mackerras, Git mailing list, patthoyts
2015-02-23 15:30 GMT-07:00 Alex Henrie <alexhenrie24@gmail.com>:
> 2015-02-16 16:27 GMT-07:00 Alex Henrie <alexhenrie24@gmail.com>:
>> 2015-02-09 14:55 GMT-07:00 Junio C Hamano <gitster@pobox.com>:
>>>
>>> Alex Henrie <alexhenrie24@gmail.com> writes:
>>>
>>> > This is just a friendly reminder that this patch has been sitting in
>>> > the mailing list archives for a couple of weeks, and it has not yet
>>> > been accepted or commented on.
>>>
>>> I think that is because the message was not sent to the right
>>> people, and also because the patch was made against a wrong project
>>> ;-).
>>>
>>> I'll forward it to the gitk maintainer after digging it out of the
>>> archive and tweaking it. Thanks.
>>>
>>> Paul, comments?
>>
>> Another week and still no comments on either this patch or the gitk
>> Catalan translation patch. Is Paul Mackerras still actively involved
>> in the project?
>>
>> -Alex
>
> Another week and still no response. If Paul is no longer maintaining
> gitk, another maintainer should be appointed.
>
> -Alex
I am unsubscribing from the Git mailing list. As far as I can tell,
the gitk project does not accept patches, not even translations into
new languages. If this policy ever changes, please email me directly.
-Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it
2015-02-09 21:55 ` Junio C Hamano
2015-02-16 23:27 ` Alex Henrie
@ 2015-03-22 3:40 ` Paul Mackerras
1 sibling, 0 replies; 7+ messages in thread
From: Paul Mackerras @ 2015-03-22 3:40 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Git mailing list, patthoyts, Alex Henrie
On Mon, Feb 09, 2015 at 01:55:23PM -0800, Junio C Hamano wrote:
> Alex Henrie <alexhenrie24@gmail.com> writes:
>
> > This is just a friendly reminder that this patch has been sitting in
> > the mailing list archives for a couple of weeks, and it has not yet
> > been accepted or commented on.
>
> I think that is because the message was not sent to the right
> people, and also because the patch was made against a wrong project
> ;-).
>
> I'll forward it to the gitk maintainer after digging it out of the
> archive and tweaking it. Thanks.
>
> Paul, comments?
>
> -- >8 --
> From: Alex Henrie <alexhenrie24@gmail.com>
> Date: Thu, 22 Jan 2015 01:19:39 -0700
> Subject: gitk: Remove tcl-format flag from a message that shouldn't have it
>
> xgettext sees "% o" and interprets it as a placeholder for an octal
> number preceded by a space. However, in this case it's not actually a
> placeholder, and most translations will replace the "% o" sequence with
> something else. Removing the tcl-format flag from this string prevents
> tools like Poedit from freaking out when "% o" doesn't appear in the
> translated string.
>
> The corrected flag will appear in each translation's po file the next time
> the translation is updated with `make update-po`.
>
> Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>
Thanks, applied.
Paul.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-03-22 3:41 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-22 8:19 [PATCH] gitk: Remove tcl-format flag from a message that shouldn't have it Alex Henrie
2015-02-09 21:36 ` Alex Henrie
2015-02-09 21:55 ` Junio C Hamano
2015-02-16 23:27 ` Alex Henrie
2015-02-23 22:30 ` Alex Henrie
2015-03-10 18:53 ` Alex Henrie
2015-03-22 3:40 ` Paul Mackerras
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.