Re: [PATCH 1/3] po/README: document PO helper
- From
Jiang Xin <worldhello.net@gmail.com>
- Date
- May 13, 2021, 15:40 UTC
- Message-ID
- <CANYiYbEVyUc6gsjb4=+8x8m64zpkTo4vYSLAB+au-Rz5xsEVaA@mail.gmail.com>
- In-Reply-To
- <2600d078-9ba0-2ebc-4611-c414ef8b12dd@gmail.com>
Bagas Sanjaya <bagasdotme@gmail.com> 于2021年3月12日周五 下午7:57写道:
Show 60 quoted lines
> > > 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