Re: pg_upgrade and PGPORT - Mailing list pgsql-hackers

From Robert Haas
Subject Re: pg_upgrade and PGPORT
Date
Msg-id BANLkTi=hTHUV7MP+Rx57-ta9X9Sqh_gFPw@mail.gmail.com
Whole thread Raw
In response to Re: pg_upgrade and PGPORT  (Peter Eisentraut <peter_e@gmx.net>)
Responses Re: pg_upgrade and PGPORT  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
On Thu, May 12, 2011 at 1:22 AM, Peter Eisentraut <peter_e@gmx.net> wrote:
> On ons, 2011-05-11 at 18:36 -0400, Bruce Momjian wrote:
>> Robert Haas wrote:
>> > On Wed, May 11, 2011 at 2:18 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> > > Or you could just "unsetenv" instead of complaining.
>> >
>> > +1 for that.
>>
>> OK, the attached patch does this, but allows PGCLIENTENCODING to be
>> passed in.  The new output looks like:
>>
>>       Performing Consistency Checks
>>       -----------------------------
>>       ignoring libpq environment variable PGPORT
>
> I haven't tried it, but I suppose option.c will now make use of PGPORT
> and then later you get that message that it was ignored?

Either way, it hardly seems necessary to emit a log message stating
that you are unsetting an environment variable.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: "Albe Laurenz"
Date:
Subject: Uninitialized SSL values? (was: Fix for bug in ldapServiceLookup in libpq)
Next
From: Tom Lane
Date:
Subject: Re: Infinity bsearch crash on Windows