git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* [PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator
@ 2021-03-11 12:55 Bagas Sanjaya
  2021-03-11 12:55 ` [PATCH 1/3] po/README: document PO helper Bagas Sanjaya
                   ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-11 12:55 UTC (permalink / raw)
  To: git; +Cc: worldhello.net, Bagas Sanjaya

Document various topics mentioned by l10n coordinator (Jiang Xin) when
pulling l10n contributions.

Bagas Sanjaya (3):
  po/README: document PO helper
  po/README: document "core translation"
  po/README: document l10n conventions

 po/README | 79 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 79 insertions(+)

-- 
2.25.1


^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 1/3] po/README: document PO helper
  2021-03-11 12:55 [PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator Bagas Sanjaya
@ 2021-03-11 12:55 ` Bagas Sanjaya
  2021-03-12  1:32   ` Jiang Xin
  2021-03-11 12:55 ` [PATCH 2/3] po/README: document "core translation" Bagas Sanjaya
  2021-03-11 12:55 ` [PATCH 3/3] po/README: document l10n conventions Bagas Sanjaya
  2 siblings, 1 reply; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-11 12:55 UTC (permalink / raw)
  To: git; +Cc: worldhello.net, Bagas Sanjaya

Document about PO helper script (po-helper.sh). It covers about
installing the script and short usage examples.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 po/README | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/po/README b/po/README
index efd5baaf1d..9beffc2954 100644
--- a/po/README
+++ b/po/README
@@ -286,3 +286,25 @@ Testing marked strings
 
 Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
 changed to account for translations as they're added.
+
+
+PO Helper
+---------
+
+To make maintaining XX.po file easier, the l10n coordinator created
+po-helper.sh script. It is wrapper to gettext suite, specifically
+written for the purpose of Git l10n workflow.
+
+To install the script, checkout "po-helper" branch, then copy
+utils/po-helper.sh to somewhere on $PATH, and mark it executable.
+
+Below are the usage examples:
+
+  - To start new language translation:
+        po-helper.sh init XX.po
+  - To update PO file:
+        po-helper.sh update XX.po
+  - To syntax check:
+        po-helper.sh check XX.po
+
+Run po-helper.sh without arguments for usage help.
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 2/3] po/README: document "core translation"
  2021-03-11 12:55 [PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator Bagas Sanjaya
  2021-03-11 12:55 ` [PATCH 1/3] po/README: document PO helper Bagas Sanjaya
@ 2021-03-11 12:55 ` Bagas Sanjaya
  2021-03-12  1:32   ` Jiang Xin
  2021-03-11 12:55 ` [PATCH 3/3] po/README: document l10n conventions Bagas Sanjaya
  2 siblings, 1 reply; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-11 12:55 UTC (permalink / raw)
  To: git; +Cc: worldhello.net, Bagas Sanjaya

We now expect new language translations to have core subset of 5000+
messages translated for these to be merged. If it hasn't been done yet,
do so by first generate core PO, then do translation work there, merge
it back to main PO (XX.po), and fix any "fuzzy" errors.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 po/README | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/po/README b/po/README
index 9beffc2954..eda7edf4e6 100644
--- a/po/README
+++ b/po/README
@@ -39,6 +39,42 @@ coordinate among yourselves and nominate the team leader for your
 language, so that the l10n coordinator only needs to interact with one
 person per language.
 
+Because there are now over 5000 messages that needs to be translated,
+we expect to complete translation for core subset of messages before
+a new language translation can be merged. See "Core translation"
+below.
+
+
+Core translation
+----------------
+The core translation file, core-XX.po, covers some of commonly used
+commands:
+
+  - wt-status.c
+  - builtin/clone.c
+  - builtin/checkout.c
+  - builtin/index-pack.c
+  - builtin/push.c
+  - builtin/reset.c
+
+To generate core-XX.po, install the helper script (see "PO Helper")
+and check the main PO file:
+
+    cd po/
+    po-helper.sh check id.po
+
+This will generate po/core-XX.po and corresponding po/core-XX.pot
+template. Now complete the translation in po/core-XX.po.
+
+When it is completed, merge core translation back to main PO file:
+
+    msgcat core-XX.po XX.po -s -o /tmp/XX.po
+    mv /tmp/XX.po XX.po
+    msgmerge --add-location --backup=off -U XX.po git.pot
+
+Fix any "fuzzy" errors that occurs, such as misplaced translated
+messages and duplicate messages.
+
 
 Translation Process Flow
 ------------------------
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [PATCH 3/3] po/README: document l10n conventions
  2021-03-11 12:55 [PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator Bagas Sanjaya
  2021-03-11 12:55 ` [PATCH 1/3] po/README: document PO helper Bagas Sanjaya
  2021-03-11 12:55 ` [PATCH 2/3] po/README: document "core translation" Bagas Sanjaya
@ 2021-03-11 12:55 ` Bagas Sanjaya
  2021-03-11 13:13   ` Bagas Sanjaya
  2021-03-12  1:33   ` Jiang Xin
  2 siblings, 2 replies; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-11 12:55 UTC (permalink / raw)
  To: git; +Cc: worldhello.net, Bagas Sanjaya

Document things that l10n contributors must folow.

Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
---
 po/README | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/po/README b/po/README
index eda7edf4e6..e1a0494b17 100644
--- a/po/README
+++ b/po/README
@@ -344,3 +344,24 @@ Below are the usage examples:
         po-helper.sh check XX.po
 
 Run po-helper.sh without arguments for usage help.
+
+
+Conventions
+-----------
+
+There are some conventions that l10n contributors must follow:
+
+   1. The subject of each commit should be prefixed with `l10n: `.
+   2. The length of commit log title (subject) should be less than 50
+      characters, and the length of other lines (commit log body)
+      should be no more than 72 characters.
+   3. Add "Signed-off-by" trailer to your commit log, like other
+      commits in Git. You can automatically add the trailer by
+      committing with `git commit -s`.
+   4. Check syntax with `msgfmt` or `po-helper.sh check` before
+      committing.
+   5. Squash trivial commits to make history clear.
+   6. Do NOT edit files outside po/ directory.
+   7. Other subsystems ("git-gui", "gitk", and Git itself) have their
+      own workflow. See Documentation/SubmittingPatches if you want
+      to make contributions to these subsystems.
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/3] po/README: document l10n conventions
  2021-03-11 12:55 ` [PATCH 3/3] po/README: document l10n conventions Bagas Sanjaya
@ 2021-03-11 13:13   ` Bagas Sanjaya
  2021-03-12  1:33   ` Jiang Xin
  1 sibling, 0 replies; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-11 13:13 UTC (permalink / raw)
  To: Bagas Sanjaya; +Cc: git

On 11/03/21 19.55, Bagas Sanjaya wrote:
> Document things that l10n contributors must folow.
> 
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>   po/README | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)

Oh oh oh, typo there. folow -> follow.

> diff --git a/po/README b/po/README
> index eda7edf4e6..e1a0494b17 100644
> --- a/po/README
> +++ b/po/README
> @@ -344,3 +344,24 @@ Below are the usage examples:
>           po-helper.sh check XX.po
>   
>   Run po-helper.sh without arguments for usage help.
> +
> +
> +Conventions
> +-----------
> +
> +There are some conventions that l10n contributors must follow:
> +
> +   1. The subject of each commit should be prefixed with `l10n: `.
> +   2. The length of commit log title (subject) should be less than 50
> +      characters, and the length of other lines (commit log body)
> +      should be no more than 72 characters.
> +   3. Add "Signed-off-by" trailer to your commit log, like other
> +      commits in Git. You can automatically add the trailer by
> +      committing with `git commit -s`.
> +   4. Check syntax with `msgfmt` or `po-helper.sh check` before
> +      committing.
> +   5. Squash trivial commits to make history clear.
> +   6. Do NOT edit files outside po/ directory.
> +   7. Other subsystems ("git-gui", "gitk", and Git itself) have their
> +      own workflow. See Documentation/SubmittingPatches if you want
> +      to make contributions to these subsystems.
> 

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/3] po/README: document PO helper
  2021-03-11 12:55 ` [PATCH 1/3] po/README: document PO helper Bagas Sanjaya
@ 2021-03-12  1:32   ` Jiang Xin
  2021-03-12  6:03     ` Junio C Hamano
  0 siblings, 1 reply; 13+ messages in thread
From: Jiang Xin @ 2021-03-12  1:32 UTC (permalink / raw)
  To: Bagas Sanjaya; +Cc: Git List

Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>
> Document about PO helper script (po-helper.sh). It covers about
> installing the script and short usage examples.
>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>  po/README | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/po/README b/po/README
> index efd5baaf1d..9beffc2954 100644
> --- a/po/README
> +++ b/po/README
> @@ -286,3 +286,25 @@ Testing marked strings
>
>  Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
>  changed to account for translations as they're added.
> +
> +
> +PO Helper
> +---------
> +
> +To make maintaining XX.po file easier, the l10n coordinator created
> +po-helper.sh script. It is wrapper to gettext suite, specifically

It's better to rename the script to other name without the suffix
".sh", so we can reimplement it in other programming language. Maybe
we can rename it to `git-po-helper`, and host this helper in a
separate project on GitHub.


> +written for the purpose of Git l10n workflow.
> +
> +To install the script, checkout "po-helper" branch, then copy
> +utils/po-helper.sh to somewhere on $PATH, and mark it executable.
> +
> +Below are the usage examples:
> +
> +  - To start new language translation:
> +        po-helper.sh init XX.po
> +  - To update PO file:
> +        po-helper.sh update XX.po
> +  - To syntax check:
> +        po-helper.sh check XX.po
> +
> +Run po-helper.sh without arguments for usage help.
> --
> 2.25.1
>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 2/3] po/README: document "core translation"
  2021-03-11 12:55 ` [PATCH 2/3] po/README: document "core translation" Bagas Sanjaya
@ 2021-03-12  1:32   ` Jiang Xin
  0 siblings, 0 replies; 13+ messages in thread
From: Jiang Xin @ 2021-03-12  1:32 UTC (permalink / raw)
  To: Bagas Sanjaya; +Cc: Git List

Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>
> We now expect new language translations to have core subset of 5000+
> messages translated for these to be merged. If it hasn't been done yet,
> do so by first generate core PO, then do translation work there, merge
> it back to main PO (XX.po), and fix any "fuzzy" errors.
>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>  po/README | 36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
>
> diff --git a/po/README b/po/README
> index 9beffc2954..eda7edf4e6 100644
> --- a/po/README
> +++ b/po/README
> @@ -39,6 +39,42 @@ coordinate among yourselves and nominate the team leader for your
>  language, so that the l10n coordinator only needs to interact with one
>  person per language.
>
> +Because there are now over 5000 messages that needs to be translated,
> +we expect to complete translation for core subset of messages before
> +a new language translation can be merged. See "Core translation"
> +below.
> +
> +
> +Core translation
> +----------------
> +The core translation file, core-XX.po, covers some of commonly used
> +commands:
> +
> +  - wt-status.c
> +  - builtin/clone.c
> +  - builtin/checkout.c
> +  - builtin/index-pack.c
> +  - builtin/push.c
> +  - builtin/reset.c
> +
> +To generate core-XX.po, install the helper script (see "PO Helper")
> +and check the main PO file:
> +
> +    cd po/
> +    po-helper.sh check id.po

s/id.po/XX.po/


> +
> +This will generate po/core-XX.po and corresponding po/core-XX.pot
> +template. Now complete the translation in po/core-XX.po.
> +
> +When it is completed, merge core translation back to main PO file:
> +
> +    msgcat core-XX.po XX.po -s -o /tmp/XX.po
> +    mv /tmp/XX.po XX.po
> +    msgmerge --add-location --backup=off -U XX.po git.pot
> +
> +Fix any "fuzzy" errors that occurs, such as misplaced translated
> +messages and duplicate messages.
> +
>
>  Translation Process Flow
>  ------------------------
> --
> 2.25.1
>

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/3] po/README: document l10n conventions
  2021-03-11 12:55 ` [PATCH 3/3] po/README: document l10n conventions Bagas Sanjaya
  2021-03-11 13:13   ` Bagas Sanjaya
@ 2021-03-12  1:33   ` Jiang Xin
  2021-03-12 11:47     ` Bagas Sanjaya
  1 sibling, 1 reply; 13+ messages in thread
From: Jiang Xin @ 2021-03-12  1:33 UTC (permalink / raw)
  To: Bagas Sanjaya; +Cc: Git List

Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>
> Document things that l10n contributors must folow.
>
> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>  po/README | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)
>
> diff --git a/po/README b/po/README
> index eda7edf4e6..e1a0494b17 100644
> --- a/po/README
> +++ b/po/README
> @@ -344,3 +344,24 @@ Below are the usage examples:
>          po-helper.sh check XX.po
>
>  Run po-helper.sh without arguments for usage help.
> +
> +
> +Conventions
> +-----------
> +
> +There are some conventions that l10n contributors must follow:
> +
> +   1. The subject of each commit should be prefixed with `l10n: `.
> +   2. The length of commit log title (subject) should be less than 50
> +      characters, and the length of other lines (commit log body)
> +      should be no more than 72 characters.
> +   3. Add "Signed-off-by" trailer to your commit log, like other
> +      commits in Git. You can automatically add the trailer by
> +      committing with `git commit -s`.
> +   4. Check syntax with `msgfmt` or `po-helper.sh check` before
> +      committing.
> +   5. Squash trivial commits to make history clear.
> +   6. Do NOT edit files outside po/ directory.
> +   7. Other subsystems ("git-gui", "gitk", and Git itself) have their
> +      own workflow. See Documentation/SubmittingPatches if you want
> +      to make contributions to these subsystems.

This patch 3/3 can be merged before we reimplement the utility `git-po-helper`.

--
Jiang Xin

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/3] po/README: document PO helper
  2021-03-12  1:32   ` Jiang Xin
@ 2021-03-12  6:03     ` Junio C Hamano
  2021-03-12 11:56       ` Bagas Sanjaya
  0 siblings, 1 reply; 13+ messages in thread
From: Junio C Hamano @ 2021-03-12  6:03 UTC (permalink / raw)
  To: Jiang Xin; +Cc: Bagas Sanjaya, Git List

Jiang Xin <worldhello.net@gmail.com> writes:

> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>>
>> Document about PO helper script (po-helper.sh). It covers about
>> installing the script and short usage examples.
>>
>> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
>> ---
>>  po/README | 22 ++++++++++++++++++++++
>>  1 file changed, 22 insertions(+)
>>
>> diff --git a/po/README b/po/README
>> index efd5baaf1d..9beffc2954 100644
>> --- a/po/README
>> +++ b/po/README
>> @@ -286,3 +286,25 @@ Testing marked strings
>>
>>  Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
>>  changed to account for translations as they're added.
>> +
>> +
>> +PO Helper
>> +---------
>> +
>> +To make maintaining XX.po file easier, the l10n coordinator created
>> +po-helper.sh script. It is wrapper to gettext suite, specifically
>
> It's better to rename the script to other name without the suffix
> ".sh", so we can reimplement it in other programming language. Maybe
> we can rename it to `git-po-helper`, and host this helper in a
> separate project on GitHub.

It might be a good move in the longer term, but if the po-helper.sh
is working well enough to fill the need of i18n/l10n team right now,
I think documenting the status quo would be a good way to help the
contributors immediately.

And when we reimplement it (if that is desirable---I have no opinion
myself), we certainly would want to update this part of the doc to
use the name of the rewritten tool.

In any case, I expect that you to be the main reviewer on this topic
and you will feed me the final commits, just like any other change
in the po/* area via a pull request.  In other words, I may have
input to the discussion on list, but I won't be applying the result
myself---I'll be pulling from git://github.com/git-l10n/git-po.git
repository instead.

Thanks.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 3/3] po/README: document l10n conventions
  2021-03-12  1:33   ` Jiang Xin
@ 2021-03-12 11:47     ` Bagas Sanjaya
  0 siblings, 0 replies; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-12 11:47 UTC (permalink / raw)
  To: Jiang Xin; +Cc: git

On 12/03/21 08.33, Jiang Xin wrote:
> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>>
>> Document things that l10n contributors must folow.
>>
>> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
>> ---
>>   po/README | 21 +++++++++++++++++++++
>>   1 file changed, 21 insertions(+)
>>
>> diff --git a/po/README b/po/README
>> index eda7edf4e6..e1a0494b17 100644
>> --- a/po/README
>> +++ b/po/README
>> @@ -344,3 +344,24 @@ Below are the usage examples:
>>           po-helper.sh check XX.po
>>
>>   Run po-helper.sh without arguments for usage help.
>> +
>> +
>> +Conventions
>> +-----------
>> +
>> +There are some conventions that l10n contributors must follow:
>> +
>> +   1. The subject of each commit should be prefixed with `l10n: `.
>> +   2. The length of commit log title (subject) should be less than 50
>> +      characters, and the length of other lines (commit log body)
>> +      should be no more than 72 characters.
>> +   3. Add "Signed-off-by" trailer to your commit log, like other
>> +      commits in Git. You can automatically add the trailer by
>> +      committing with `git commit -s`.
>> +   4. Check syntax with `msgfmt` or `po-helper.sh check` before
>> +      committing.
>> +   5. Squash trivial commits to make history clear.
>> +   6. Do NOT edit files outside po/ directory.
>> +   7. Other subsystems ("git-gui", "gitk", and Git itself) have their
>> +      own workflow. See Documentation/SubmittingPatches if you want
>> +      to make contributions to these subsystems.
> 
> This patch 3/3 can be merged before we reimplement the utility `git-po-helper`.
> 
> --
> Jiang Xin
> 
Did you mean this patch can be sent separated from this series?

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/3] po/README: document PO helper
  2021-03-12  6:03     ` Junio C Hamano
@ 2021-03-12 11:56       ` Bagas Sanjaya
  2021-05-13 15:40         ` Jiang Xin
  0 siblings, 1 reply; 13+ messages in thread
From: Bagas Sanjaya @ 2021-03-12 11:56 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git


On 12/03/21 13.03, Junio C Hamano wrote:
> Jiang Xin <worldhello.net@gmail.com> writes:
> 
>> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
>>>
>>> Document about PO helper script (po-helper.sh). It covers about
>>> installing the script and short usage examples.
>>>
>>> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
>>> ---
>>>   po/README | 22 ++++++++++++++++++++++
>>>   1 file changed, 22 insertions(+)
>>>
>>> diff --git a/po/README b/po/README
>>> index efd5baaf1d..9beffc2954 100644
>>> --- a/po/README
>>> +++ b/po/README
>>> @@ -286,3 +286,25 @@ Testing marked strings
>>>
>>>   Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
>>>   changed to account for translations as they're added.
>>> +
>>> +
>>> +PO Helper
>>> +---------
>>> +
>>> +To make maintaining XX.po file easier, the l10n coordinator created
>>> +po-helper.sh script. It is wrapper to gettext suite, specifically
>>
>> It's better to rename the script to other name without the suffix
>> ".sh", so we can reimplement it in other programming language. Maybe
>> we can rename it to `git-po-helper`, and host this helper in a
>> separate project on GitHub.
> 
> It might be a good move in the longer term, but if the po-helper.sh
> is working well enough to fill the need of i18n/l10n team right now,
> I think documenting the status quo would be a good way to help the
> contributors immediately.
> 
> And when we reimplement it (if that is desirable---I have no opinion
> myself), we certainly would want to update this part of the doc to
> use the name of the rewritten tool.
> 
> In any case, I expect that you to be the main reviewer on this topic
> and you will feed me the final commits, just like any other change
> in the po/* area via a pull request.  In other words, I may have
> input to the discussion on list, but I won't be applying the result
> myself---I'll be pulling from git://github.com/git-l10n/git-po.git
> repository instead.
> 
> Thanks.
> 

Grazie Junio. However, I'm more of writing documentation rather than
contributing code, so I will help for the documentation and usage
testing. But anyways, we can ask for help from other l10n contributors
who are more proficient at coding.

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/3] po/README: document PO helper
  2021-03-12 11:56       ` Bagas Sanjaya
@ 2021-05-13 15:40         ` Jiang Xin
  2021-05-27  3:11           ` Jiang Xin
  0 siblings, 1 reply; 13+ messages in thread
From: Jiang Xin @ 2021-05-13 15:40 UTC (permalink / raw)
  To: Bagas Sanjaya
  Cc: Junio C Hamano, Git List, Alexander Shopov, Jordi Mas,
	Matthias Rüster, Jimmy Angelakos, Christopher Díaz,
	Jean-Noël Avila, Alessandro Menti, Gwan-gyeong Mun, Arusekk,
	Daniel Santos, Dimitriy Ryazantcev, Peter Krefting, Emir SARI,
	Trần Ngọc Quân, Jiang Xin, Yi-Jyun Pan

Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月12日周五 下午7:57写道:
>
>
> On 12/03/21 13.03, Junio C Hamano wrote:
> > Jiang Xin <worldhello.net@gmail.com> writes:
> >
> >> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
> >>>
> >>> Document about PO helper script (po-helper.sh). It covers about
> >>> installing the script and short usage examples.
> >>>
> >>> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> >>> ---
> >>>   po/README | 22 ++++++++++++++++++++++
> >>>   1 file changed, 22 insertions(+)
> >>>
> >>> diff --git a/po/README b/po/README
> >>> index efd5baaf1d..9beffc2954 100644
> >>> --- a/po/README
> >>> +++ b/po/README
> >>> @@ -286,3 +286,25 @@ Testing marked strings
> >>>
> >>>   Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
> >>>   changed to account for translations as they're added.
> >>> +
> >>> +
> >>> +PO Helper
> >>> +---------
> >>> +
> >>> +To make maintaining XX.po file easier, the l10n coordinator created
> >>> +po-helper.sh script. It is wrapper to gettext suite, specifically
> >>
> >> It's better to rename the script to other name without the suffix
> >> ".sh", so we can reimplement it in other programming language. Maybe
> >> we can rename it to `git-po-helper`, and host this helper in a
> >> separate project on GitHub.
> >
> > It might be a good move in the longer term, but if the po-helper.sh
> > is working well enough to fill the need of i18n/l10n team right now,
> > I think documenting the status quo would be a good way to help the
> > contributors immediately.
> >
> > And when we reimplement it (if that is desirable---I have no opinion
> > myself), we certainly would want to update this part of the doc to
> > use the name of the rewritten tool.
> >
> > In any case, I expect that you to be the main reviewer on this topic
> > and you will feed me the final commits, just like any other change
> > in the po/* area via a pull request.  In other words, I may have
> > input to the discussion on list, but I won't be applying the result
> > myself---I'll be pulling from git://github.com/git-l10n/git-po.git
> > repository instead.
> >
> > Thanks.
> >
>
> Grazie Junio. However, I'm more of writing documentation rather than
> contributing code, so I will help for the documentation and usage
> testing. But anyways, we can ask for help from other l10n contributors
> who are more proficient at coding.
>
Hi Bagas,

I created a new repository at
https://github.com/git-l10n/git-po-helper that contains a new helper
program `git-po-helper` to replace the obsolete `po-helper.sh` script.

The new `git-po-helper` program can check character encoding of the
commit log, check author/committer datetime, and check format of
`po/TEAMS`, etc., in addition to the conventions listed in README of
the new project "git-l10n/git-po-helper".

You can update your patches according to this change.

All l10n contributors can use this new helper to check their l10n contributions.

BTW, with the help of Daniel, a google group named git-l10n is created
for discussion on git-l10n. This mail is also cc to this email
address: git-l10n@googlegroups.com .

--
Jiang Xin

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/3] po/README: document PO helper
  2021-05-13 15:40         ` Jiang Xin
@ 2021-05-27  3:11           ` Jiang Xin
  0 siblings, 0 replies; 13+ messages in thread
From: Jiang Xin @ 2021-05-27  3:11 UTC (permalink / raw)
  To: Bagas Sanjaya
  Cc: Git List, Alexander Shopov, Jordi Mas, Matthias Rüster,
	Jimmy Angelakos, Christopher Díaz, Jean-Noël Avila,
	Alessandro Menti, Gwan-gyeong Mun, Arusekk, Daniel Santos,
	Dimitriy Ryazantcev, Peter Krefting, Emir SARI,
	Trần Ngọc Quân, Yi-Jyun Pan

Jiang Xin <worldhello.net@gmail.com> 于2021年5月13日周四 下午11:40写道:
>
> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月12日周五 下午7:57写道:
> >
> >
> > On 12/03/21 13.03, Junio C Hamano wrote:
> > > Jiang Xin <worldhello.net@gmail.com> writes:
> > >
> > >> Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月11日周四 下午8:56写道:
> > >>>
> > >>> Document about PO helper script (po-helper.sh). It covers about
> > >>> installing the script and short usage examples.
> > >>>
> > >>> Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com>
> > >>> ---
> > >>>   po/README | 22 ++++++++++++++++++++++
> > >>>   1 file changed, 22 insertions(+)
> > >>>
> > >>> diff --git a/po/README b/po/README
> > >>> index efd5baaf1d..9beffc2954 100644
> > >>> --- a/po/README
> > >>> +++ b/po/README
> > >>> @@ -286,3 +286,25 @@ Testing marked strings
> > >>>
> > >>>   Git's tests are run under LANG=C LC_ALL=C. So the tests do not need be
> > >>>   changed to account for translations as they're added.
> > >>> +
> > >>> +
> > >>> +PO Helper
> > >>> +---------
> > >>> +
> > >>> +To make maintaining XX.po file easier, the l10n coordinator created
> > >>> +po-helper.sh script. It is wrapper to gettext suite, specifically
> > >>
> > >> It's better to rename the script to other name without the suffix
> > >> ".sh", so we can reimplement it in other programming language. Maybe
> > >> we can rename it to `git-po-helper`, and host this helper in a
> > >> separate project on GitHub.
> > >
> > > It might be a good move in the longer term, but if the po-helper.sh
> > > is working well enough to fill the need of i18n/l10n team right now,
> > > I think documenting the status quo would be a good way to help the
> > > contributors immediately.
> > >
> > > And when we reimplement it (if that is desirable---I have no opinion
> > > myself), we certainly would want to update this part of the doc to
> > > use the name of the rewritten tool.
> > >
> > > In any case, I expect that you to be the main reviewer on this topic
> > > and you will feed me the final commits, just like any other change
> > > in the po/* area via a pull request.  In other words, I may have
> > > input to the discussion on list, but I won't be applying the result
> > > myself---I'll be pulling from git://github.com/git-l10n/git-po.git
> > > repository instead.
> > >
> > > Thanks.
> > >
> >
> > Grazie Junio. However, I'm more of writing documentation rather than
> > contributing code, so I will help for the documentation and usage
> > testing. But anyways, we can ask for help from other l10n contributors
> > who are more proficient at coding.
> >
> Hi Bagas,
>
> I created a new repository at
> https://github.com/git-l10n/git-po-helper that contains a new helper
> program `git-po-helper` to replace the obsolete `po-helper.sh` script.
>
> The new `git-po-helper` program can check character encoding of the
> commit log, check author/committer datetime, and check format of
> `po/TEAMS`, etc., in addition to the conventions listed in README of
> the new project "git-l10n/git-po-helper".
>
> You can update your patches according to this change.
>
> All l10n contributors can use this new helper to check their l10n contributions.
>
> BTW, with the help of Daniel, a google group named git-l10n is created
> for discussion on git-l10n. This mail is also cc to this email
> address: git-l10n@googlegroups.com .

Based on pull request form Bagas, that made some nice changes for
"po/README", I recreated a new pull request #525 on GitHub:

    https://github.com/git-l10n/git-po/pull/525

Please help to review.  This pull request contains the following commits:

 + 826abe532f l10n: README: note on fuzzy translations
 + 93f2050952 l10n: README: document l10n conventions
 + 1466dc3b86 l10n: README: document "core translation"
 + 06d662e594 l10n: README: document git-po-helper
 + a5724c83a2 l10n: README: add file extention ".md"

You can download the commits using command:

    git fetch https://github.com/git-l10n/git-po refs/pull/525/head

--
Jiang Xin

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2021-05-27  3:11 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-11 12:55 [PATCH 0/3] po/README: document miscellaneous topics mentioned by l10n coordinator Bagas Sanjaya
2021-03-11 12:55 ` [PATCH 1/3] po/README: document PO helper Bagas Sanjaya
2021-03-12  1:32   ` Jiang Xin
2021-03-12  6:03     ` Junio C Hamano
2021-03-12 11:56       ` Bagas Sanjaya
2021-05-13 15:40         ` Jiang Xin
2021-05-27  3:11           ` Jiang Xin
2021-03-11 12:55 ` [PATCH 2/3] po/README: document "core translation" Bagas Sanjaya
2021-03-12  1:32   ` Jiang Xin
2021-03-11 12:55 ` [PATCH 3/3] po/README: document l10n conventions Bagas Sanjaya
2021-03-11 13:13   ` Bagas Sanjaya
2021-03-12  1:33   ` Jiang Xin
2021-03-12 11:47     ` Bagas Sanjaya

Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).