Horton

Thread

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

  1. Add new files to nls.mk and add translation markers

  1. pgsql: Add new files to nls.mk and add translation markers

    Peter Eisentraut <peter_e@gmx.net> — 2017-08-03T02:46:12Z

    Add new files to nls.mk and add translation markers

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5ff3d73813ebcc3ff80be77c30b458d728951036

Modified Files
--------------
src/bin/pg_basebackup/nls.mk       | 4 ++--
src/bin/pg_basebackup/walmethods.c | 2 +-
src/interfaces/libpq/nls.mk        | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)

  2. fixing pg_upgrade strings (was Re: pgsql: Add new files to nls.mk and add translation)

    Alvaro Herrera <alvherre@2ndquadrant.com> — 2017-08-03T03:47:08Z

    Peter Eisentraut wrote:
> Add new files to nls.mk and add translation markers

This reminds me that I noticed a few days ago another really serious
broken piece in pg_upgrade where check_required_directory() is incurring
in the ugliest case of string building I've ever seen.  I didn't have
the courage to try to fix it back then, but I think we should dedicate
it some time.  It does this:

            pg_fatal("You must identify the directory where the %s.\n"
                     "Please use the %s command-line option or the %s environment variable.\n",
                     description, cmdLineOption, envVarName);

and the callsites are like this:

    check_required_directory(&new_cluster.bindir, NULL, "PGBINNEW", "-B",
                             _("new cluster binaries reside"));
    check_required_directory(&old_cluster.pgdata, &old_cluster.pgconfig,
                             "PGDATAOLD", "-d", _("old cluster data resides"));

note the declensions don't match even in the English original.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

  3. Re: fixing pg_upgrade strings (was Re: pgsql: Add new files to nls.mk and add translation)

    Tom Lane <tgl@sss.pgh.pa.us> — 2017-08-03T04:15:52Z

    Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> It does this:

>             pg_fatal("You must identify the directory where the %s.\n"
>                      "Please use the %s command-line option or the %s environment variable.\n",
>                      description, cmdLineOption, envVarName);

> and the callsites are like this:

>     check_required_directory(&new_cluster.bindir, NULL, "PGBINNEW", "-B",
>                              _("new cluster binaries reside"));
>     check_required_directory(&old_cluster.pgdata, &old_cluster.pgconfig,
>                              "PGDATAOLD", "-d", _("old cluster data resides"));

> note the declensions don't match even in the English original.

FWIW, I think the English declensions are fine as long as you consider
"data" to be a group noun --- "data reside" would read a bit funny IMO.
But certainly this is an utter translatability fail.  Maybe pass a boolean
to indicate one of two first sentences to use?  Or actually, might as well
consider the entire message string to be one of two translatable options.
It's not like we have a need to indicate more than one option or envar
name for each case.

			regards, tom lane