Re: partial pg_regress patch from Magnus

From: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
To: Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: partial pg_regress patch from Magnus
Date: 2007-01-19 14:13:36
Message-ID: 20070119141336.GD26080@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

Alvaro Herrera wrote:
> Alvaro Herrera wrote:
>
> > I tested it in a VPATH and a normal build, and regression test still
> > pass. This is mostly Magnus' code: I merely stripped out the parts
> > unrelated to this change, and reworked the code a little. So the
> > possible bugs are likely mine.
>
> Woops, it doesn't work in VPATH :-( Sorry, I'll fix and resubmit.
> (It only worked before because the parallel test had created the files,
> and I failed to run "make clean" between runs).

It seems the problem was only that the makefile was neglecting to add
the --srcdir argument. It was kind of annoying that it didn't show any
warning or anything; I spent quite some time trying to figure out what
was going wrong. I think I'll add a check that if it detects less than,
say, 5 files, throw an error message and quit.

Other than that, if I don't see objections, I'll commit it later today.

--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Roman Nowak 2007-01-19 14:50:24 BUG #2913: Subscript on multidimensional array yields no value
Previous Message Bruce Momjian 2007-01-19 13:56:48 Re: [HACKERS] [PATCHES] [BUGS] BUG #2846: inconsistent and