Re: [PATCH 1/3] po/README: document PO helper

4 messages, 3 authors, 2021-05-27 · open the first message on its own page

Re: [PATCH 1/3] po/README: document PO helper

From: Junio C Hamano <hidden>
Date: 2021-03-12 06:04:30

Jiang Xin [off-list ref] writes:
Bagas Sanjaya [off-list ref] 于2021年3月11日周四 下午8:56写道:
quoted
Document about PO helper script (po-helper.sh). It covers about
installing the script and short usage examples.

Signed-off-by: Bagas Sanjaya <redacted>
---
 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.

Re: [PATCH 1/3] po/README: document PO helper

From: Bagas Sanjaya <hidden>
Date: 2021-03-12 11:57:27

On 12/03/21 13.03, Junio C Hamano wrote:
Jiang Xin [off-list ref] writes:
quoted
Bagas Sanjaya [off-list ref] 于2021年3月11日周四 下午8:56写道:
quoted
Document about PO helper script (po-helper.sh). It covers about
installing the script and short usage examples.

Signed-off-by: Bagas Sanjaya <redacted>
---
  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

Re: [PATCH 1/3] po/README: document PO helper

From: Jiang Xin <hidden>
Date: 2021-05-13 15:40:26

Bagas Sanjaya [off-list ref] 于2021年3月12日周五 下午7:57写道:

On 12/03/21 13.03, Junio C Hamano wrote:
quoted
Jiang Xin [off-list ref] writes:
quoted
Bagas Sanjaya [off-list ref] 于2021年3月11日周四 下午8:56写道:
quoted
Document about PO helper script (po-helper.sh). It covers about
installing the script and short usage examples.

Signed-off-by: Bagas Sanjaya <redacted>
---
  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

Re: [PATCH 1/3] po/README: document PO helper

From: Jiang Xin <hidden>
Date: 2021-05-27 03:11:44

Jiang Xin [off-list ref] 于2021年5月13日周四 下午11:40写道:
Bagas Sanjaya [off-list ref] 于2021年3月12日周五 下午7:57写道:
quoted

On 12/03/21 13.03, Junio C Hamano wrote:
quoted
Jiang Xin [off-list ref] writes:
quoted
Bagas Sanjaya [off-list ref] 于2021年3月11日周四 下午8:56写道:
quoted
Document about PO helper script (po-helper.sh). It covers about
installing the script and short usage examples.

Signed-off-by: Bagas Sanjaya <redacted>
---
  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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help