Re: [COMMITTERS] pgsql: Easier to translate psql help Instead of requiring translators

Lists: pgsql-committerspgsql-hackers
From: petere(at)postgresql(dot)org (Peter Eisentraut)
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Easier to translate psql help Instead of requiring translators
Date: 2009-09-18 05:00:42
Message-ID: 20090918050042.8E9FD753FB7@cvs.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers pgsql-hackers

Log Message:
-----------
Easier to translate psql help

Instead of requiring translators to translate the entire SQL command
synopses, change create_help.pl to only require them to translate the
placeholders, and paste those into the synopsis using a printf mechanism.
Make some small updates to the markup to make it easier to parse.

Note: This causes msgmerge of gettext 0.17 to segfault. You will need
the patch from https://savannah.gnu.org/bugs/?27474 to make it work.
msgmerge usually only runs on babel.postgresql.org, however.

Modified Files:
--------------
pgsql/doc/src/sgml/ref:
alter_database.sgml (r1.22 -> r1.23)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/alter_database.sgml?r1=1.22&r2=1.23)
alter_function.sgml (r1.17 -> r1.18)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/alter_function.sgml?r1=1.17&r2=1.18)
alter_role.sgml (r1.12 -> r1.13)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/alter_role.sgml?r1=1.12&r2=1.13)
alter_table.sgml (r1.108 -> r1.109)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/alter_table.sgml?r1=1.108&r2=1.109)
alter_user.sgml (r1.45 -> r1.46)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/alter_user.sgml?r1=1.45&r2=1.46)
begin.sgml (r1.37 -> r1.38)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/begin.sgml?r1=1.37&r2=1.38)
create_domain.sgml (r1.32 -> r1.33)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_domain.sgml?r1=1.32&r2=1.33)
create_group.sgml (r1.19 -> r1.20)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_group.sgml?r1=1.19&r2=1.20)
create_role.sgml (r1.11 -> r1.12)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_role.sgml?r1=1.11&r2=1.12)
create_table.sgml (r1.115 -> r1.116)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_table.sgml?r1=1.115&r2=1.116)
create_user.sgml (r1.41 -> r1.42)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/create_user.sgml?r1=1.41&r2=1.42)
fetch.sgml (r1.43 -> r1.44)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/fetch.sgml?r1=1.43&r2=1.44)
grant.sgml (r1.75 -> r1.76)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/grant.sgml?r1=1.75&r2=1.76)
lock.sgml (r1.52 -> r1.53)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/lock.sgml?r1=1.52&r2=1.53)
select.sgml (r1.124 -> r1.125)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/select.sgml?r1=1.124&r2=1.125)
set_transaction.sgml (r1.26 -> r1.27)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/set_transaction.sgml?r1=1.26&r2=1.27)
start_transaction.sgml (r1.17 -> r1.18)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/doc/src/sgml/ref/start_transaction.sgml?r1=1.17&r2=1.18)
pgsql/src/bin/psql:
Makefile (r1.65 -> r1.66)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/Makefile?r1=1.65&r2=1.66)
create_help.pl (r1.19 -> r1.20)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/create_help.pl?r1=1.19&r2=1.20)
help.c (r1.151 -> r1.152)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/help.c?r1=1.151&r2=1.152)
nls.mk (r1.23 -> r1.24)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/bin/psql/nls.mk?r1=1.23&r2=1.24)


From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Easier to translate psql help Instead of requiring translators
Date: 2009-09-18 13:46:33
Message-ID: 15089.1253281593@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers pgsql-hackers

petere(at)postgresql(dot)org (Peter Eisentraut) writes:
> Log Message:
> -----------
> Easier to translate psql help

Looks like this broke the msvc build ...

regards, tom lane


From: Josh Williams <joshwilliams(at)ij(dot)net>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Easier to translate psql help Instead of requiring translators
Date: 2009-09-19 03:54:27
Message-ID: 1253332467.7374.7.camel@lapdragon
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers pgsql-hackers

On Fri, 2009-09-18 at 09:46 -0400, Tom Lane wrote:
> petere(at)postgresql(dot)org (Peter Eisentraut) writes:
> > Log Message:
> > -----------
> > Easier to translate psql help
>
> Looks like this broke the msvc build ...

Yep, one of create_help.pl's arguments changed to not include the file
extension, but msvc was still trying to give it "sql_help.h".
Quickpatch attached.

Wondered why my attempts to test and review another patch were suddenly
confounded...

> regards, tom lane

- Josh Williams

Attachment Content-Type Size
msvc-sql_help.diff text/x-patch 795 bytes

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Josh Williams <joshwilliams(at)ij(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [COMMITTERS] pgsql: Easier to translate psql help Instead of requiring translators
Date: 2009-09-19 05:57:36
Message-ID: 4AB472D0.5020202@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-committers pgsql-hackers

Josh Williams wrote:
> On Fri, 2009-09-18 at 09:46 -0400, Tom Lane wrote:
>
>> petere(at)postgresql(dot)org (Peter Eisentraut) writes:
>>
>>> Log Message:
>>> -----------
>>> Easier to translate psql help
>>>
>> Looks like this broke the msvc build ...
>>
>
> Yep, one of create_help.pl's arguments changed to not include the file
> extension, but msvc was still trying to give it "sql_help.h".
> Quickpatch attached.
>
>

Thanks. Applied.

cheers

andrew