Re: stupid patch of pg_dumplo

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Karel Zak <zakkr(at)zf(dot)jcu(dot)cz>
Cc: andrea gelmini <bungle(at)linux(dot)it>, pgsql-patches(at)postgresql(dot)org
Subject: Re: stupid patch of pg_dumplo
Date: 2002-02-22 20:06:46
Message-ID: 200202222006.g1MK6kZ14430@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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(at)zf(dot)jcu(dot)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(at)postgresql(dot)org
>

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

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2002-02-22 20:14:29 Re: Patch to add Heimdal kerberos support
Previous Message Bruce Momjian 2002-02-22 20:05:36 Re: stupid patch of pg_dumplo