LP#1709932: recognize more strings from oils_i18n_gettext() user/csharp/lp1709932_improve_db_seed_i18n
authorGalen Charlton <gmc@equinoxinitiative.org>
Thu, 10 Aug 2017 16:45:58 +0000 (12:45 -0400)
committerChris Sharp <csharp@georgialibraries.org>
Thu, 10 Aug 2017 20:41:58 +0000 (16:41 -0400)
commitd84932436544d38920b6060a6795cad31ba7dbf1
treed6a4e6c62e13803ac0fac47189e82855316ffdba
parenta903911387ca31e49f115f5eb2aa4605b7600aaf
LP#1709932: recognize more strings from oils_i18n_gettext()

This patch makes db-seed-i18n.py more flexible about
where commas between strings in invocations of oils_i18n_gettext()
can appear, reduces the chances that strings fail to be grabbed
for translation. It also improves the error message shown
when parsing errors occur.

To test
-------
[1] Apply patch and run

    cd build/i18n && make sql2pot

[2] Verify that no error messages are produced during
    this step.
[3] Inspect the changes to po/db.seed/db.seed.pot and
    verify that additional strings are added, including
    "Item Print Label Font Family".
[4] (optional) Deliberately remove a comma from an
    oils_i18n_gettext invocation in 950.data.seed-values.sql
    and run make sql2pot again. The error message shoudl
    now include the failing chunk of text.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
build/i18n/scripts/db-seed-i18n.py