Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres - Mailing list pgsql-committers

From Michael Paquier
Subject Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres
Date
Msg-id YKMJ0Ri2B1yysTXC@paquier.xyz
Whole thread Raw
In response to Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On Mon, May 17, 2021 at 05:51:54PM -0400, Tom Lane wrote:
> I wrote:
>> I think what we want to do is have this code invoked only in
>> test directories that explicitly ask for it, say with a new
>> "--make-testtablespace" switch for pg_regress.
>
> Say, as attached.  (Windows part is untested.)

Thanks.  I was going to produce something this morning, but you have
been faster than me.

One thing that's changing in this patch is that testtablespace would
be created even if the input directory does not exist when using
--make-testtablespace-dir.  I would have kept the creation of the
tablespace path within convert_sourcefiles_in() for this reason.
Worth noting that snprintf() is used twice instead of once to build
the tablespace path string.  The Windows part works correctly.
--
Michael

Attachment

pgsql-committers by date:

Previous
From: David Rowley
Date:
Subject: pgsql: Fix typo and outdated information in README.barrier
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Move tablespace path re-creation from the makefiles to pg_regres