Tom Lane wrote:
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
>> Tom Lane wrote:
>>> The original design had only the Makefile knowing exactly where the
>>> testtablespace directory lurks, making it changeable in one place,
>>> and also confining the whole business to src/test/regress which is good
>>> because none of the other uses of pg_regress need it. As CVS tip
>>> stands, the location of testtablespace is hard-wired into the pg_regress
>>> executable, and I think it's mostly blind accident that it doesn't
>>> clutter contrib/* and src/pl/* with testtablespace directories.
>> Commandline parameter, perhaps?
> I was trying to avoid that, but only because it'd make the patch bigger.
> It would be one way to resolve the above gripes.
> But I'm still thinking that it's fairly silly to expect Windows users to
> type out the equivalent of
> ./pg_regress --psqldir=$(PSQLDIR) --schedule=$(srcdir)/parallel_schedule --srcdir=$(abs_srcdir) --multibyte=$(MULTIBYTE) --load-language=plpgsql $(MAXCONNOPT) $(NOLOCALE)
Yeah, agreed. The original idea was to have a shortcut created by the
installer to do that for you.
> especially when no other platform is that hard. (Admittedly, some of
> these switches have usable defaults, but many don't.) So it really
> seems like the right answer is a .BAT script, and I'm kinda surprised
> to hear there's not one already.
Well, there is a way to run it from the msvc build. The idea was to be
able to run after a binary install. Granted it doesn't make that much
sense, but it was mostly a "customer feel good" thing. But there's
nothing preventing that from being done from a .BAT file instead of a
In response to
pgsql-hackers by date
|Next:||From: Bruce Momjian||Date: 2008-01-19 00:06:59|
|Subject: Traveling to London|
|Previous:||From: Tom Lane||Date: 2008-01-18 22:37:24|
|Subject: Re: Recent pg_regress changes break testing under SELinux |