Re: stupid patch of pg_dumplo - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: stupid patch of pg_dumplo
Date
Msg-id 200202222006.g1MK6kZ14430@candle.pha.pa.us
Whole thread Raw
In response to Re: stupid patch of pg_dumplo  (Karel Zak <zakkr@zf.jcu.cz>)
Responses Re: stupid patch of pg_dumplo  (andrea gelmini <andrea.gelmini@linux.it>)
List pgsql-patches
Actually, I have to modify the patch to match these requested changes.
I will do it unless someone gets to it first.

---------------------------------------------------------------------------

Karel Zak wrote:
> On Thu, Nov 08, 2001 at 01:52:47PM +0100, andrea gelmini wrote:
> > hi all,
> >      i would like to se this patch applied, because i often use
> >      pg_dumplo, and i need to set the port number of the server to contact.
>
>  It's good idea, maybe you can try add other values usable in
>  PQsetdbLogin.
>
> >      anyway, i don't have C language skill, so be careful with these
> >      lines... they work for me, but...
>
>  Please use other code in pg_dumplo/main.c as inspiration.
>
>  If you want add "port" add it to pgLO struct (see pg_dumplo.h) and
>  fill this value like others in this sources.
>
> >                   case 's':
> >                       pgLO->space = strdup(optarg);
> >                       break;
> > +                 case 'o':
> > +                     pgport = optarg;
> > +                     break;
>
>  rather:
>                    case 'o':
>                         pgLO->port = strdup(optarg);
>                         break;
>
> > !     pgLO->conn = PQsetdbLogin(pgLO->host, pgport, NULL, NULL, pgLO->db,
> >                                 pgLO->user, pwd);
>
>  rather:
>
>  pgLO->conn = PQsetdbLogin(pgLO->host, pgLO->port, NULL, NULL, pgLO->db,
>                                 pgLO->user, pwd);
>
>
>  Please correct your patch and I mean will no problem add it to
>  official code.
>
>  Thanks
>
>         Karel
>
> --
>  Karel Zak  <zakkr@zf.jcu.cz>
>  http://home.zf.jcu.cz/~zakkr/
>
>  C, PostgreSQL, PHP, WWW, http://docs.linux.cz, http://mape.jcu.cz
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo@postgresql.org
>

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: stupid patch of pg_dumplo
Next
From: Bruce Momjian
Date:
Subject: Re: Patch to add Heimdal kerberos support