Hi,
On 2018-03-18 19:30:33 -0400, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
> > On March 18, 2018 4:06:18 PM PDT, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> >> Hm ... pg_regress unsets PGDATABASE, along with the other related
> >> environment variables, when it has a temp installation but not
> >> when it doesn't. So what I don't understand is why your environment
> >> doesn't also break every other regression test besides ecpg.
>
> > All the others specify a database. The issue with the ecpg test is that
> > it doesn't for two test cases.
>
> Ah. Well, it doesn't seem unreasonable to want to test that case,
> so I don't think "remove the test case" is the right answer.
Right.
> Is it sane for pg_regress to unset PGDATABASE unconditionally? Not
> sure, but if we're generally always specifying a value, maybe that's
> OK.
I'm not sure either. I wonder whether we should just make ecpg's
pg_regress invocation do so? That seems to be the way of least
resistance ;)
Greetings,
Andres Freund
