pgsql: Make pg_regress.c unset PGDATABASE during make installcheck. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Make pg_regress.c unset PGDATABASE during make installcheck.
Date
Msg-id E1iDyaT-0007nJ-CD@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make pg_regress.c unset PGDATABASE during make installcheck.

For the most part, we leave libpq-controlling environment variables
alone during "make installcheck", reasoning that connecting to the
server the user expects us to connect to may depend on those variables.
But that argument doesn't apply to PGDATABASE, since we always want
to connect to a specific database name within the server.  And failing
to unset it causes certain ECPG tests to fail, as various people have
complained of in the past.  So let's unset it.

Possibly this should be back-patched, but I'm disinclined to do that
right before 12.0 release.  Maybe later.

Discussion: https://postgr.es/m/20180318205548.2akxjqvo7hrk5wbc@alap3.anarazel.de
Discussion: https://postgr.es/m/E1bOum4-0002EA-2y@gemulon.postgresql.org

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5ee96b3e2221d154ffcb719bd2dee1179c53f821

Modified Files
--------------
src/test/regress/pg_regress.c | 8 ++++++++
1 file changed, 8 insertions(+)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: Re: [COMMITTERS] pgsql: Establish conventions about global objectnames used in regressi
Next
From: Noah Misch
Date:
Subject: Re: pgsql: Test pg_atomic_fetch_add_ with variable addend and 16-bitedge c