Re: pg_ctl-related "fixes" - Mailing list pgsql-committers

From Serguei A. Mokhov
Subject Re: pg_ctl-related "fixes"
Date
Msg-id Pine.LNX.4.44.0406011631470.17520-100000@haida.cs.concordia.ca
Whole thread Raw
In response to Re: pg_ctl-related "fixes"  (Bruce Momjian <pgman@candle.pha.pa.us>)
Responses Re: pg_ctl-related "fixes"
List pgsql-committers
On Tue, 1 Jun 2004, Bruce Momjian wrote:

> Date: Tue, 1 Jun 2004 16:19:05 -0400 (EDT)
>
> > > The first commit just did the init_nls changes while the second was all
> > > pg_ctl.  Sorry.
> >
> > But the locale and NLS issues for pg_ctl still remain unfixed.
>
> The only thing I have not done is to add the Russian part.  pg_ctl.c
> should have everthing it needs, right?

It's missing these entirely:

***************
*** 798,803 ****
--- 799,810 ----
  int
  main(int argc, char **argv)
  {
+ #ifdef ENABLE_NLS
+     setlocale(LC_ALL, "");
+     bindtextdomain(progname, LOCALEDIR);
+     textdomain(progname);
+ #endif
+
      static struct option long_options[] = {
          {"help", no_argument, NULL, '?'},
          {"version", no_argument, NULL, 'V'},


http://archives.postgresql.org/pgsql-patches/2004-05/msg00499.php

Actually, as I look at the code now, the above additions should go after

        progname = get_progname(argv[0]);

Maybe I missed something while doing the diff to 1.1 and then it all
drifted under me.

--
Serguei A. Mokhov            |  /~\    The ASCII
Computer Science Department  |  \ / Ribbon Campaign
Concordia University         |   X    Against HTML
Montreal, Quebec, Canada     |  / \      Email!


pgsql-committers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: pg_ctl-related "fixes"
Next
From: tgl@svr1.postgresql.org (Tom Lane)
Date:
Subject: pgsql-server: Align GRANT/REVOKE behavior more closely with the SQL