Re: [patch] PSQLDIR not passed to pg_regress in contrib/pg_upgrade/test.sh

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Christoph Berg <cb(at)df7cb(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [patch] PSQLDIR not passed to pg_regress in contrib/pg_upgrade/test.sh
Date: 2013-05-10 19:10:28
Message-ID: CA+TgmoYe1nS4aeSUCBqdB6wHXL+cTFeYatkkwK3=T1MFe+XS0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, May 7, 2013 at 2:40 AM, Christoph Berg <cb(at)df7cb(dot)de> wrote:
> "make check-world" in 9.3beta1 fails if you don't actually have 9.3
> installed. In contrib/pg_upgrade/test.sh, it will try pg_regress
> --psqldir=/usr/lib/postgresql/9.3/bin which doesn't exist.

I tried to reproduce this and couldn't. The following already-exstant
chunk of code seems to be trying to cover the same issue:

# We need to make it use psql from our temporary installation,
# because otherwise the installcheck run below would try to
# use psql from the proper installation directory, which might
# be outdated or missing. But don't override anything else that's
# already in EXTRA_REGRESS_OPTS.
EXTRA_REGRESS_OPTS="$EXTRA_REGRESS_OPTS --psqldir=$bindir"
export EXTRA_REGRESS_OPTS

If I comment out those last two lines, then I can reproduce the problem exactly.

I wonder why it's working differently for you.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2013-05-10 19:21:05 Re: corrupt pages detected by enabling checksums
Previous Message Alvaro Herrera 2013-05-10 18:55:40 Re: Bug in VACUUM reporting of "removed %d row versions" in 9.2+