Re: pgsql: Reduce risk of accidentally running temp-install regression tests

From: Michael Meskes <meskes(at)postgresql(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Peter Eisentraut <petere(at)postgresql(dot)org>, Michael Meskes <meskes(at)postgresql(dot)org>, pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Reduce risk of accidentally running temp-install regression tests
Date: 2008-11-30 14:21:44
Message-ID: 20081130142144.GA2652@feivel.credativ.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Fri, Nov 28, 2008 at 06:54:15PM -0500, Tom Lane wrote:
> temp port. But it looks like the temp port number is actually wired
> into some of the ecpg tests' expected results, and so getting rid of it
> is not as easy as one could wish.

Should only one test, that some installations do not even run, but still, yes,
it makes things more difficult.

> Michael, could you look at removing that dependency so we can let
> pg_regress.c select the port number as it wishes? If it's not practical
> to suppress the port number in the regression test output, maybe things
> could be changed so that pg_regress.c itself substitutes in the port
> number it's chosen.

Suppressing the output defies the purpose for this one test, namely to check
different ways to connect. If pg_regress is able do the job of two sed calls in
can easily substitute the port number itself. I'm not sure if this is worth the
hassle, or whether we just remove these parts of the regression test.

Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes(at)jabber(dot)org
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message User Dim 2008-11-30 16:40:40 prefix - prefix: WIP: explicit initialiser function prefix_range()
Previous Message User Cunha17 2008-11-30 03:08:51 snapshot - src: Added support for missing datatypes.