{"thread":{"id":"55290","subject":"[PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator","startedAt":"2021-03-11T12:57:15Z","lastAt":"2021-05-27T03:11:44Z","messageCount":13,"participants":["Bagas Sanjaya","Jiang Xin","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":3},"messages":[{"id":"418839","messageId":"20210311125511.51152-1-bagasdotme@gmail.com","threadId":"55290","inReplyTo":null,"subject":"[PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-11T12:55:08Z","receivedAt":"2021-03-11T12:57:15Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"Document various topics mentioned by l10n coordinator (Jiang Xin) when\npulling l10n contributions.\n\nBagas Sanjaya (3):\n po/README: document PO helper\n po/README: document \"core translation\"\n po/README: document l10n conventions\n\n po/README | 79 +++++++++++++++++++++++++++++++++++++++++++++++++++++++\n 1 file changed, 79 insertions(+)\n\n-- \n2.25.1\n\n"},{"id":"418840","messageId":"20210311125511.51152-4-bagasdotme@gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-1-bagasdotme@gmail.com","subject":"[PATCH 3/3] po/README: document l10n conventions","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-11T12:55:11Z","receivedAt":"2021-03-11T12:57:19Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"Document things that l10n contributors must folow.\n\nSigned-off-by: Bagas Sanjaya \n---\n po/README | 21 +++++++++++++++++++++\n 1 file changed, 21 insertions(+)\n\ndiff --git a/po/README b/po/README\nindex eda7edf4e6..e1a0494b17 100644\n--- a/po/README\n+++ b/po/README\n@@ -344,3 +344,24 @@ Below are the usage examples:\n po-helper.sh check XX.po\n \n Run po-helper.sh without arguments for usage help.\n+\n+\n+Conventions\n+-----------\n+\n+There are some conventions that l10n contributors must follow:\n+\n+ 1. The subject of each commit should be prefixed with `l10n: `.\n+ 2. The length of commit log title (subject) should be less than 50\n+ characters, and the length of other lines (commit log body)\n+ should be no more than 72 characters.\n+ 3. Add \"Signed-off-by\" trailer to your commit log, like other\n+ commits in Git. You can automatically add the trailer by\n+ committing with `git commit -s`.\n+ 4. Check syntax with `msgfmt` or `po-helper.sh check` before\n+ committing.\n+ 5. Squash trivial commits to make history clear.\n+ 6. Do NOT edit files outside po/ directory.\n+ 7. Other subsystems (\"git-gui\", \"gitk\", and Git itself) have their\n+ own workflow. See Documentation/SubmittingPatches if you want\n+ to make contributions to these subsystems.\n-- \n2.25.1\n\n"},{"id":"418841","messageId":"20210311125511.51152-2-bagasdotme@gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-1-bagasdotme@gmail.com","subject":"[PATCH 1/3] po/README: document PO helper","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-11T12:55:09Z","receivedAt":"2021-03-11T12:57:19Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"Document about PO helper script (po-helper.sh). It covers about\ninstalling the script and short usage examples.\n\nSigned-off-by: Bagas Sanjaya \n---\n po/README | 22 ++++++++++++++++++++++\n 1 file changed, 22 insertions(+)\n\ndiff --git a/po/README b/po/README\nindex efd5baaf1d..9beffc2954 100644\n--- a/po/README\n+++ b/po/README\n@@ -286,3 +286,25 @@ Testing marked strings\n \n Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n changed to account for translations as they're added.\n+\n+\n+PO Helper\n+---------\n+\n+To make maintaining XX.po file easier, the l10n coordinator created\n+po-helper.sh script. It is wrapper to gettext suite, specifically\n+written for the purpose of Git l10n workflow.\n+\n+To install the script, checkout \"po-helper\" branch, then copy\n+utils/po-helper.sh to somewhere on $PATH, and mark it executable.\n+\n+Below are the usage examples:\n+\n+ - To start new language translation:\n+ po-helper.sh init XX.po\n+ - To update PO file:\n+ po-helper.sh update XX.po\n+ - To syntax check:\n+ po-helper.sh check XX.po\n+\n+Run po-helper.sh without arguments for usage help.\n-- \n2.25.1\n\n"},{"id":"418842","messageId":"20210311125511.51152-3-bagasdotme@gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-1-bagasdotme@gmail.com","subject":"[PATCH 2/3] po/README: document \"core translation\"","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-11T12:55:10Z","receivedAt":"2021-03-11T12:57:19Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"We now expect new language translations to have core subset of 5000+\nmessages translated for these to be merged. If it hasn't been done yet,\ndo so by first generate core PO, then do translation work there, merge\nit back to main PO (XX.po), and fix any \"fuzzy\" errors.\n\nSigned-off-by: Bagas Sanjaya \n---\n po/README | 36 ++++++++++++++++++++++++++++++++++++\n 1 file changed, 36 insertions(+)\n\ndiff --git a/po/README b/po/README\nindex 9beffc2954..eda7edf4e6 100644\n--- a/po/README\n+++ b/po/README\n@@ -39,6 +39,42 @@ coordinate among yourselves and nominate the team leader for your\n language, so that the l10n coordinator only needs to interact with one\n person per language.\n \n+Because there are now over 5000 messages that needs to be translated,\n+we expect to complete translation for core subset of messages before\n+a new language translation can be merged. See \"Core translation\"\n+below.\n+\n+\n+Core translation\n+----------------\n+The core translation file, core-XX.po, covers some of commonly used\n+commands:\n+\n+ - wt-status.c\n+ - builtin/clone.c\n+ - builtin/checkout.c\n+ - builtin/index-pack.c\n+ - builtin/push.c\n+ - builtin/reset.c\n+\n+To generate core-XX.po, install the helper script (see \"PO Helper\")\n+and check the main PO file:\n+\n+ cd po/\n+ po-helper.sh check id.po\n+\n+This will generate po/core-XX.po and corresponding po/core-XX.pot\n+template. Now complete the translation in po/core-XX.po.\n+\n+When it is completed, merge core translation back to main PO file:\n+\n+ msgcat core-XX.po XX.po -s -o /tmp/XX.po\n+ mv /tmp/XX.po XX.po\n+ msgmerge --add-location --backup=off -U XX.po git.pot\n+\n+Fix any \"fuzzy\" errors that occurs, such as misplaced translated\n+messages and duplicate messages.\n+\n \n Translation Process Flow\n ------------------------\n-- \n2.25.1\n\n"},{"id":"418844","messageId":"9b50b5e1-33c2-2562-4b3a-f25db3f6ac1c@gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-4-bagasdotme@gmail.com","subject":"Re: [PATCH 3/3] po/README: document l10n conventions","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-11T13:13:01Z","receivedAt":"2021-03-11T13:13:49Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 11/03/21 19.55, Bagas Sanjaya wrote:\n> Document things that l10n contributors must folow.\n> \n> Signed-off-by: Bagas Sanjaya \n> ---\n> po/README | 21 +++++++++++++++++++++\n> 1 file changed, 21 insertions(+)\n\nOh oh oh, typo there. folow -> follow.\n\n> diff --git a/po/README b/po/README\n> index eda7edf4e6..e1a0494b17 100644\n> --- a/po/README\n> +++ b/po/README\n> @@ -344,3 +344,24 @@ Below are the usage examples:\n> po-helper.sh check XX.po\n> \n> Run po-helper.sh without arguments for usage help.\n> +\n> +\n> +Conventions\n> +-----------\n> +\n> +There are some conventions that l10n contributors must follow:\n> +\n> + 1. The subject of each commit should be prefixed with `l10n: `.\n> + 2. The length of commit log title (subject) should be less than 50\n> + characters, and the length of other lines (commit log body)\n> + should be no more than 72 characters.\n> + 3. Add \"Signed-off-by\" trailer to your commit log, like other\n> + commits in Git. You can automatically add the trailer by\n> + committing with `git commit -s`.\n> + 4. Check syntax with `msgfmt` or `po-helper.sh check` before\n> + committing.\n> + 5. Squash trivial commits to make history clear.\n> + 6. Do NOT edit files outside po/ directory.\n> + 7. Other subsystems (\"git-gui\", \"gitk\", and Git itself) have their\n> + own workflow. See Documentation/SubmittingPatches if you want\n> + to make contributions to these subsystems.\n> \n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"418916","messageId":"CANYiYbFDJ-CDsJs04=uPsUvcVqPA8wpY2g8ZP_6R5UsYQbQvAg@mail.gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-4-bagasdotme@gmail.com","subject":"Re: [PATCH 3/3] po/README: document l10n conventions","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2021-03-12T01:33:02Z","receivedAt":"2021-03-12T01:34:03Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>\n> Document things that l10n contributors must folow.\n>\n> Signed-off-by: Bagas Sanjaya \n> ---\n> po/README | 21 +++++++++++++++++++++\n> 1 file changed, 21 insertions(+)\n>\n> diff --git a/po/README b/po/README\n> index eda7edf4e6..e1a0494b17 100644\n> --- a/po/README\n> +++ b/po/README\n> @@ -344,3 +344,24 @@ Below are the usage examples:\n> po-helper.sh check XX.po\n>\n> Run po-helper.sh without arguments for usage help.\n> +\n> +\n> +Conventions\n> +-----------\n> +\n> +There are some conventions that l10n contributors must follow:\n> +\n> + 1. The subject of each commit should be prefixed with `l10n: `.\n> + 2. The length of commit log title (subject) should be less than 50\n> + characters, and the length of other lines (commit log body)\n> + should be no more than 72 characters.\n> + 3. Add \"Signed-off-by\" trailer to your commit log, like other\n> + commits in Git. You can automatically add the trailer by\n> + committing with `git commit -s`.\n> + 4. Check syntax with `msgfmt` or `po-helper.sh check` before\n> + committing.\n> + 5. Squash trivial commits to make history clear.\n> + 6. Do NOT edit files outside po/ directory.\n> + 7. Other subsystems (\"git-gui\", \"gitk\", and Git itself) have their\n> + own workflow. See Documentation/SubmittingPatches if you want\n> + to make contributions to these subsystems.\n\nThis patch 3/3 can be merged before we reimplement the utility `git-po-helper`.\n\n--\nJiang Xin\n"},{"id":"418917","messageId":"CANYiYbE-Pr7LSOB-qjLn0jus-ERzUXR7E3LTWb8_35THaqA1FQ@mail.gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-3-bagasdotme@gmail.com","subject":"Re: [PATCH 2/3] po/README: document \"core translation\"","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2021-03-12T01:32:49Z","receivedAt":"2021-03-12T01:34:04Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>\n> We now expect new language translations to have core subset of 5000+\n> messages translated for these to be merged. If it hasn't been done yet,\n> do so by first generate core PO, then do translation work there, merge\n> it back to main PO (XX.po), and fix any \"fuzzy\" errors.\n>\n> Signed-off-by: Bagas Sanjaya \n> ---\n> po/README | 36 ++++++++++++++++++++++++++++++++++++\n> 1 file changed, 36 insertions(+)\n>\n> diff --git a/po/README b/po/README\n> index 9beffc2954..eda7edf4e6 100644\n> --- a/po/README\n> +++ b/po/README\n> @@ -39,6 +39,42 @@ coordinate among yourselves and nominate the team leader for your\n> language, so that the l10n coordinator only needs to interact with one\n> person per language.\n>\n> +Because there are now over 5000 messages that needs to be translated,\n> +we expect to complete translation for core subset of messages before\n> +a new language translation can be merged. See \"Core translation\"\n> +below.\n> +\n> +\n> +Core translation\n> +----------------\n> +The core translation file, core-XX.po, covers some of commonly used\n> +commands:\n> +\n> + - wt-status.c\n> + - builtin/clone.c\n> + - builtin/checkout.c\n> + - builtin/index-pack.c\n> + - builtin/push.c\n> + - builtin/reset.c\n> +\n> +To generate core-XX.po, install the helper script (see \"PO Helper\")\n> +and check the main PO file:\n> +\n> + cd po/\n> + po-helper.sh check id.po\n\ns/id.po/XX.po/\n\n\n> +\n> +This will generate po/core-XX.po and corresponding po/core-XX.pot\n> +template. Now complete the translation in po/core-XX.po.\n> +\n> +When it is completed, merge core translation back to main PO file:\n> +\n> + msgcat core-XX.po XX.po -s -o /tmp/XX.po\n> + mv /tmp/XX.po XX.po\n> + msgmerge --add-location --backup=off -U XX.po git.pot\n> +\n> +Fix any \"fuzzy\" errors that occurs, such as misplaced translated\n> +messages and duplicate messages.\n> +\n>\n> Translation Process Flow\n> ------------------------\n> --\n> 2.25.1\n>\n"},{"id":"418918","messageId":"CANYiYbH_U=3gN+LH7e-LBdRdcE0geAuUzoBX4O09qego0xKX-Q@mail.gmail.com","threadId":"55290","inReplyTo":"20210311125511.51152-2-bagasdotme@gmail.com","subject":"Re: [PATCH 1/3] po/README: document PO helper","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2021-03-12T01:32:39Z","receivedAt":"2021-03-12T01:34:04Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>\n> Document about PO helper script (po-helper.sh). It covers about\n> installing the script and short usage examples.\n>\n> Signed-off-by: Bagas Sanjaya \n> ---\n> po/README | 22 ++++++++++++++++++++++\n> 1 file changed, 22 insertions(+)\n>\n> diff --git a/po/README b/po/README\n> index efd5baaf1d..9beffc2954 100644\n> --- a/po/README\n> +++ b/po/README\n> @@ -286,3 +286,25 @@ Testing marked strings\n>\n> Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n> changed to account for translations as they're added.\n> +\n> +\n> +PO Helper\n> +---------\n> +\n> +To make maintaining XX.po file easier, the l10n coordinator created\n> +po-helper.sh script. It is wrapper to gettext suite, specifically\n\nIt's better to rename the script to other name without the suffix\n\".sh\", so we can reimplement it in other programming language. Maybe\nwe can rename it to `git-po-helper`, and host this helper in a\nseparate project on GitHub.\n\n\n> +written for the purpose of Git l10n workflow.\n> +\n> +To install the script, checkout \"po-helper\" branch, then copy\n> +utils/po-helper.sh to somewhere on $PATH, and mark it executable.\n> +\n> +Below are the usage examples:\n> +\n> + - To start new language translation:\n> + po-helper.sh init XX.po\n> + - To update PO file:\n> + po-helper.sh update XX.po\n> + - To syntax check:\n> + po-helper.sh check XX.po\n> +\n> +Run po-helper.sh without arguments for usage help.\n> --\n> 2.25.1\n>\n"},{"id":"418930","messageId":"xmqqa6r8c3i0.fsf@gitster.g","threadId":"55290","inReplyTo":"CANYiYbH_U=3gN+LH7e-LBdRdcE0geAuUzoBX4O09qego0xKX-Q@mail.gmail.com","subject":"Re: [PATCH 1/3] po/README: document PO helper","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-03-12T06:03:51Z","receivedAt":"2021-03-12T06:04:30Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jiang Xin writes:\n\n> Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>>\n>> Document about PO helper script (po-helper.sh). It covers about\n>> installing the script and short usage examples.\n>>\n>> Signed-off-by: Bagas Sanjaya \n>> ---\n>> po/README | 22 ++++++++++++++++++++++\n>> 1 file changed, 22 insertions(+)\n>>\n>> diff --git a/po/README b/po/README\n>> index efd5baaf1d..9beffc2954 100644\n>> --- a/po/README\n>> +++ b/po/README\n>> @@ -286,3 +286,25 @@ Testing marked strings\n>>\n>> Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n>> changed to account for translations as they're added.\n>> +\n>> +\n>> +PO Helper\n>> +---------\n>> +\n>> +To make maintaining XX.po file easier, the l10n coordinator created\n>> +po-helper.sh script. It is wrapper to gettext suite, specifically\n>\n> It's better to rename the script to other name without the suffix\n> \".sh\", so we can reimplement it in other programming language. Maybe\n> we can rename it to `git-po-helper`, and host this helper in a\n> separate project on GitHub.\n\nIt might be a good move in the longer term, but if the po-helper.sh\nis working well enough to fill the need of i18n/l10n team right now,\nI think documenting the status quo would be a good way to help the\ncontributors immediately.\n\nAnd when we reimplement it (if that is desirable---I have no opinion\nmyself), we certainly would want to update this part of the doc to\nuse the name of the rewritten tool.\n\nIn any case, I expect that you to be the main reviewer on this topic\nand you will feed me the final commits, just like any other change\nin the po/* area via a pull request. In other words, I may have\ninput to the discussion on list, but I won't be applying the result\nmyself---I'll be pulling from git://github.com/git-l10n/git-po.git\nrepository instead.\n\nThanks.\n"},{"id":"418965","messageId":"8b3f1c87-0cae-0c5c-78ec-1167bd89230f@gmail.com","threadId":"55290","inReplyTo":"CANYiYbFDJ-CDsJs04=uPsUvcVqPA8wpY2g8ZP_6R5UsYQbQvAg@mail.gmail.com","subject":"Re: [PATCH 3/3] po/README: document l10n conventions","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-12T11:47:50Z","receivedAt":"2021-03-12T11:48:56Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"On 12/03/21 08.33, Jiang Xin wrote:\n> Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>>\n>> Document things that l10n contributors must folow.\n>>\n>> Signed-off-by: Bagas Sanjaya \n>> ---\n>> po/README | 21 +++++++++++++++++++++\n>> 1 file changed, 21 insertions(+)\n>>\n>> diff --git a/po/README b/po/README\n>> index eda7edf4e6..e1a0494b17 100644\n>> --- a/po/README\n>> +++ b/po/README\n>> @@ -344,3 +344,24 @@ Below are the usage examples:\n>> po-helper.sh check XX.po\n>>\n>> Run po-helper.sh without arguments for usage help.\n>> +\n>> +\n>> +Conventions\n>> +-----------\n>> +\n>> +There are some conventions that l10n contributors must follow:\n>> +\n>> + 1. The subject of each commit should be prefixed with `l10n: `.\n>> + 2. The length of commit log title (subject) should be less than 50\n>> + characters, and the length of other lines (commit log body)\n>> + should be no more than 72 characters.\n>> + 3. Add \"Signed-off-by\" trailer to your commit log, like other\n>> + commits in Git. You can automatically add the trailer by\n>> + committing with `git commit -s`.\n>> + 4. Check syntax with `msgfmt` or `po-helper.sh check` before\n>> + committing.\n>> + 5. Squash trivial commits to make history clear.\n>> + 6. Do NOT edit files outside po/ directory.\n>> + 7. Other subsystems (\"git-gui\", \"gitk\", and Git itself) have their\n>> + own workflow. See Documentation/SubmittingPatches if you want\n>> + to make contributions to these subsystems.\n> \n> This patch 3/3 can be merged before we reimplement the utility `git-po-helper`.\n> \n> --\n> Jiang Xin\n> \nDid you mean this patch can be sent separated from this series?\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"418966","messageId":"2600d078-9ba0-2ebc-4611-c414ef8b12dd@gmail.com","threadId":"55290","inReplyTo":"xmqqa6r8c3i0.fsf@gitster.g","subject":"Re: [PATCH 1/3] po/README: document PO helper","fromName":"Bagas Sanjaya","fromEmail":"bagasdotme@gmail.com","sentAt":"2021-03-12T11:56:27Z","receivedAt":"2021-03-12T11:57:27Z","isPatch":true,"sender":{"key":"bagasdotme@gmail.com","avatar":"https://avatars.githubusercontent.com/u/40219486?v=4"},"body":"\nOn 12/03/21 13.03, Junio C Hamano wrote:\n> Jiang Xin writes:\n> \n>> Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n>>>\n>>> Document about PO helper script (po-helper.sh). It covers about\n>>> installing the script and short usage examples.\n>>>\n>>> Signed-off-by: Bagas Sanjaya \n>>> ---\n>>> po/README | 22 ++++++++++++++++++++++\n>>> 1 file changed, 22 insertions(+)\n>>>\n>>> diff --git a/po/README b/po/README\n>>> index efd5baaf1d..9beffc2954 100644\n>>> --- a/po/README\n>>> +++ b/po/README\n>>> @@ -286,3 +286,25 @@ Testing marked strings\n>>>\n>>> Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n>>> changed to account for translations as they're added.\n>>> +\n>>> +\n>>> +PO Helper\n>>> +---------\n>>> +\n>>> +To make maintaining XX.po file easier, the l10n coordinator created\n>>> +po-helper.sh script. It is wrapper to gettext suite, specifically\n>>\n>> It's better to rename the script to other name without the suffix\n>> \".sh\", so we can reimplement it in other programming language. Maybe\n>> we can rename it to `git-po-helper`, and host this helper in a\n>> separate project on GitHub.\n> \n> It might be a good move in the longer term, but if the po-helper.sh\n> is working well enough to fill the need of i18n/l10n team right now,\n> I think documenting the status quo would be a good way to help the\n> contributors immediately.\n> \n> And when we reimplement it (if that is desirable---I have no opinion\n> myself), we certainly would want to update this part of the doc to\n> use the name of the rewritten tool.\n> \n> In any case, I expect that you to be the main reviewer on this topic\n> and you will feed me the final commits, just like any other change\n> in the po/* area via a pull request. In other words, I may have\n> input to the discussion on list, but I won't be applying the result\n> myself---I'll be pulling from git://github.com/git-l10n/git-po.git\n> repository instead.\n> \n> Thanks.\n> \n\nGrazie Junio. However, I'm more of writing documentation rather than\ncontributing code, so I will help for the documentation and usage\ntesting. But anyways, we can ask for help from other l10n contributors\nwho are more proficient at coding.\n\n-- \nAn old man doll... just what I always wanted! - Clara\n"},{"id":"424483","messageId":"CANYiYbEVyUc6gsjb4=+8x8m64zpkTo4vYSLAB+au-Rz5xsEVaA@mail.gmail.com","threadId":"55290","inReplyTo":"2600d078-9ba0-2ebc-4611-c414ef8b12dd@gmail.com","subject":"Re: [PATCH 1/3] po/README: document PO helper","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2021-05-13T15:40:11Z","receivedAt":"2021-05-13T15:40:26Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"Bagas Sanjaya 于2021年3月12日周五 下午7:57写道：\n>\n>\n> On 12/03/21 13.03, Junio C Hamano wrote:\n> > Jiang Xin writes:\n> >\n> >> Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n> >>>\n> >>> Document about PO helper script (po-helper.sh). It covers about\n> >>> installing the script and short usage examples.\n> >>>\n> >>> Signed-off-by: Bagas Sanjaya \n> >>> ---\n> >>> po/README | 22 ++++++++++++++++++++++\n> >>> 1 file changed, 22 insertions(+)\n> >>>\n> >>> diff --git a/po/README b/po/README\n> >>> index efd5baaf1d..9beffc2954 100644\n> >>> --- a/po/README\n> >>> +++ b/po/README\n> >>> @@ -286,3 +286,25 @@ Testing marked strings\n> >>>\n> >>> Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n> >>> changed to account for translations as they're added.\n> >>> +\n> >>> +\n> >>> +PO Helper\n> >>> +---------\n> >>> +\n> >>> +To make maintaining XX.po file easier, the l10n coordinator created\n> >>> +po-helper.sh script. It is wrapper to gettext suite, specifically\n> >>\n> >> It's better to rename the script to other name without the suffix\n> >> \".sh\", so we can reimplement it in other programming language. Maybe\n> >> we can rename it to `git-po-helper`, and host this helper in a\n> >> separate project on GitHub.\n> >\n> > It might be a good move in the longer term, but if the po-helper.sh\n> > is working well enough to fill the need of i18n/l10n team right now,\n> > I think documenting the status quo would be a good way to help the\n> > contributors immediately.\n> >\n> > And when we reimplement it (if that is desirable---I have no opinion\n> > myself), we certainly would want to update this part of the doc to\n> > use the name of the rewritten tool.\n> >\n> > In any case, I expect that you to be the main reviewer on this topic\n> > and you will feed me the final commits, just like any other change\n> > in the po/* area via a pull request. In other words, I may have\n> > input to the discussion on list, but I won't be applying the result\n> > myself---I'll be pulling from git://github.com/git-l10n/git-po.git\n> > repository instead.\n> >\n> > Thanks.\n> >\n>\n> Grazie Junio. However, I'm more of writing documentation rather than\n> contributing code, so I will help for the documentation and usage\n> testing. But anyways, we can ask for help from other l10n contributors\n> who are more proficient at coding.\n>\nHi Bagas,\n\nI created a new repository at\nhttps://github.com/git-l10n/git-po-helper that contains a new helper\nprogram `git-po-helper` to replace the obsolete `po-helper.sh` script.\n\nThe new `git-po-helper` program can check character encoding of the\ncommit log, check author/committer datetime, and check format of\n`po/TEAMS`, etc., in addition to the conventions listed in README of\nthe new project \"git-l10n/git-po-helper\".\n\nYou can update your patches according to this change.\n\nAll l10n contributors can use this new helper to check their l10n contributions.\n\nBTW, with the help of Daniel, a google group named git-l10n is created\nfor discussion on git-l10n. This mail is also cc to this email\naddress: git-l10n@googlegroups.com .\n\n--\nJiang Xin\n"},{"id":"425649","messageId":"CANYiYbE5re95ovud4-37QVp+cmBnS0kYksM0TeCcuAX0wv+qbA@mail.gmail.com","threadId":"55290","inReplyTo":"CANYiYbEVyUc6gsjb4=+8x8m64zpkTo4vYSLAB+au-Rz5xsEVaA@mail.gmail.com","subject":"Re: [PATCH 1/3] po/README: document PO helper","fromName":"Jiang Xin","fromEmail":"worldhello.net@gmail.com","sentAt":"2021-05-27T03:11:28Z","receivedAt":"2021-05-27T03:11:44Z","isPatch":true,"sender":{"key":"worldhello.net@gmail.com","avatar":"https://avatars.githubusercontent.com/u/183860?v=4"},"body":"Jiang Xin 于2021年5月13日周四 下午11:40写道：\n>\n> Bagas Sanjaya 于2021年3月12日周五 下午7:57写道：\n> >\n> >\n> > On 12/03/21 13.03, Junio C Hamano wrote:\n> > > Jiang Xin writes:\n> > >\n> > >> Bagas Sanjaya 于2021年3月11日周四 下午8:56写道：\n> > >>>\n> > >>> Document about PO helper script (po-helper.sh). It covers about\n> > >>> installing the script and short usage examples.\n> > >>>\n> > >>> Signed-off-by: Bagas Sanjaya \n> > >>> ---\n> > >>> po/README | 22 ++++++++++++++++++++++\n> > >>> 1 file changed, 22 insertions(+)\n> > >>>\n> > >>> diff --git a/po/README b/po/README\n> > >>> index efd5baaf1d..9beffc2954 100644\n> > >>> --- a/po/README\n> > >>> +++ b/po/README\n> > >>> @@ -286,3 +286,25 @@ Testing marked strings\n> > >>>\n> > >>> Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be\n> > >>> changed to account for translations as they're added.\n> > >>> +\n> > >>> +\n> > >>> +PO Helper\n> > >>> +---------\n> > >>> +\n> > >>> +To make maintaining XX.po file easier, the l10n coordinator created\n> > >>> +po-helper.sh script. It is wrapper to gettext suite, specifically\n> > >>\n> > >> It's better to rename the script to other name without the suffix\n> > >> \".sh\", so we can reimplement it in other programming language. Maybe\n> > >> we can rename it to `git-po-helper`, and host this helper in a\n> > >> separate project on GitHub.\n> > >\n> > > It might be a good move in the longer term, but if the po-helper.sh\n> > > is working well enough to fill the need of i18n/l10n team right now,\n> > > I think documenting the status quo would be a good way to help the\n> > > contributors immediately.\n> > >\n> > > And when we reimplement it (if that is desirable---I have no opinion\n> > > myself), we certainly would want to update this part of the doc to\n> > > use the name of the rewritten tool.\n> > >\n> > > In any case, I expect that you to be the main reviewer on this topic\n> > > and you will feed me the final commits, just like any other change\n> > > in the po/* area via a pull request. In other words, I may have\n> > > input to the discussion on list, but I won't be applying the result\n> > > myself---I'll be pulling from git://github.com/git-l10n/git-po.git\n> > > repository instead.\n> > >\n> > > Thanks.\n> > >\n> >\n> > Grazie Junio. However, I'm more of writing documentation rather than\n> > contributing code, so I will help for the documentation and usage\n> > testing. But anyways, we can ask for help from other l10n contributors\n> > who are more proficient at coding.\n> >\n> Hi Bagas,\n>\n> I created a new repository at\n> https://github.com/git-l10n/git-po-helper that contains a new helper\n> program `git-po-helper` to replace the obsolete `po-helper.sh` script.\n>\n> The new `git-po-helper` program can check character encoding of the\n> commit log, check author/committer datetime, and check format of\n> `po/TEAMS`, etc., in addition to the conventions listed in README of\n> the new project \"git-l10n/git-po-helper\".\n>\n> You can update your patches according to this change.\n>\n> All l10n contributors can use this new helper to check their l10n contributions.\n>\n> BTW, with the help of Daniel, a google group named git-l10n is created\n> for discussion on git-l10n. This mail is also cc to this email\n> address: git-l10n@googlegroups.com .\n\nBased on pull request form Bagas, that made some nice changes for\n\"po/README\", I recreated a new pull request #525 on GitHub:\n\n https://github.com/git-l10n/git-po/pull/525\n\nPlease help to review. This pull request contains the following commits:\n\n + 826abe532f l10n: README: note on fuzzy translations\n + 93f2050952 l10n: README: document l10n conventions\n + 1466dc3b86 l10n: README: document \"core translation\"\n + 06d662e594 l10n: README: document git-po-helper\n + a5724c83a2 l10n: README: add file extention \".md\"\n\nYou can download the commits using command:\n\n git fetch https://github.com/git-l10n/git-po refs/pull/525/head\n\n--\nJiang Xin\n"}]}