Thread

  1. request patch pg_recvlogical.c, pg_receivexlog.c for nls

    Ioseph Kim <pgsql-kr@postgresql.kr> — 2015-12-28T08:07:22Z

    Hello,
    
    at PostgreSQL version 9.5rc1,
    at line 934 in pg_recvlogical.c 
    
            set_pglocale_pgservice(argv[0],PG_TEXTDOMAIN("pg_recvlogical"));
    
    this references invalid catalog file, because pg_basebackup,
    pg_recvlogical, pg_receivexlog commands use same catalog file
    pg_basebackup.mo
    
    pg_receivexlog.c too.
    
    patch please. 
    
    
    
    
    
    
  2. Re: request patch pg_recvlogical.c, pg_receivexlog.c for nls

    Michael Paquier <michael.paquier@gmail.com> — 2015-12-28T13:33:23Z

    On Mon, Dec 28, 2015 at 5:07 PM, Ioseph Kim <pgsql-kr@postgresql.kr> wrote:
    > Hello,
    >
    > at PostgreSQL version 9.5rc1,
    > at line 934 in pg_recvlogical.c
    >
    >         set_pglocale_pgservice(argv[0],PG_TEXTDOMAIN("pg_recvlogical"));
    >
    > this references invalid catalog file, because pg_basebackup,
    > pg_recvlogical, pg_receivexlog commands use same catalog file
    > pg_basebackup.mo
    >
    > pg_receivexlog.c too.
    >
    > patch please.
    
    Yes, you are right. Both values do not match CATALOG_NAME in nls.mk. I
    guess that something like the attached is fine then... The other
    binaries are fine.
    -- 
    Michael
    
  3. Re: request patch pg_recvlogical.c, pg_receivexlog.c for nls

    Alvaro Herrera <alvherre@2ndquadrant.com> — 2015-12-28T13:54:09Z

    Michael Paquier wrote:
    > On Mon, Dec 28, 2015 at 5:07 PM, Ioseph Kim <pgsql-kr@postgresql.kr> wrote:
    
    > > at PostgreSQL version 9.5rc1,
    > > at line 934 in pg_recvlogical.c
    > >
    > >         set_pglocale_pgservice(argv[0],PG_TEXTDOMAIN("pg_recvlogical"));
    > >
    > > this references invalid catalog file, because pg_basebackup,
    > > pg_recvlogical, pg_receivexlog commands use same catalog file
    > > pg_basebackup.mo
    > >
    > > pg_receivexlog.c too.
    > >
    > > patch please.
    > 
    > Yes, you are right. Both values do not match CATALOG_NAME in nls.mk. I
    > guess that something like the attached is fine then... The other
    > binaries are fine.
    
    Oops.  We should have noticed this a long ago, since pg_receivexlog was
    introduced in 9.2 and pg_recvlogical in 9.4 ... judging by the
    misalignment in the spanish translation for --help, I have never tested
    the output of those programs.
    
    Pushed and backpatched as appropriate.  Thanks.
    
    -- 
    Álvaro Herrera                http://www.2ndQuadrant.com/
    PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services