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

[Jiang Xin <worldhello.net@gmail.com>]

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