Message-ID: <55B90161.5090506@iki.fi>
Date: Wed, 29 Jul 2015 19:37:53 +0300
From: Heikki Linnakangas <hlinnaka@iki.fi>
User-Agent: Mozilla/5.0 (X11; Linux x86_64;
 rv:31.0) Gecko/20100101 Icedove/31.7.0
MIME-Version: 1.0
To: Michael Paquier <michael.paquier@gmail.com>,
 Noah Misch <noah@leadboat.com>
CC: PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Subject: Re: Supporting TAP tests with MSVC and Windows
References: 
 <CAB7nPqTQwphkDfZP07w7yBnbFNDhW_JBAMyCFAkarE2VWg8irQ@mail.gmail.com>
	<20150403062230.GA1153652@tornado.leadboat.com>
	<CAB7nPqSKRM0TvY3qFrGVEQpqJuaNstufG6LdD_5FMe7iSD686Q@mail.gmail.com>
	<CAB7nPqTMmanGhAZC0GME_HyFTSgOqvKYu8rF3uAP6pnL+f0QNA@mail.gmail.com>
	<CAB7nPqSHObwq89SydPYWWBafXEm=6qcxskU1-OqQOR3Zf8PrCw@mail.gmail.com>
	<CAB7nPqSmLEkGvu10ttT1bpgiT_5Hsnef6+6TNfFU_r2T1OPL4A@mail.gmail.com>
	<CAB7nPqSYzUDTj25XvYuTbzMSdCNEwo4-ofmXjXuJipqdiOVrwQ@mail.gmail.com>
	<CAB7nPqRvsJ7WvgNDQk+yFZGNUbAbMi-LRCwgcWaAcfV3R7V2nw@mail.gmail.com>
	<CAB7nPqSmCHA726iMtVuZZ4t0Rd4jkqZhDf52dTv4138iDXy7aQ@mail.gmail.com>
	<55B2758E.1010205@iki.fi>	<20150725071408.GA1442444@tornado.leadboat.com>
	<CAB7nPqRYn4_AA270BJ8VXryGV_9mzeLMTvdC2LhXJk0CxJx2+A@mail.gmail.com>
 <CAB7nPqR4YxKOSW3zUNLwEexNqOLLFtCVWb0cTF1VKx6q53=pAA@mail.gmail.com>
In-Reply-To: 
 <CAB7nPqR4YxKOSW3zUNLwEexNqOLLFtCVWb0cTF1VKx6q53=pAA@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

On 07/29/2015 10:13 AM, Michael Paquier wrote:
> On Sat, Jul 25, 2015 at 10:54 PM, Michael Paquier
> <michael.paquier@gmail.com> wrote:
>> An updated patch is attached.
>
> Attached is v9, that fixes conflicts with 01f6bb4 and recent commits
> that added TAP tests in pg_basebackup series.

Thanks, committed with some more tweaking. Peter just added a 
slurp_file() function to TestLib.pm, so I used that to replace the call 
to 'cat' instead of my previous snippet. I also fixed the issue in the 
pg_basebackup test, that LSN is not necessarily 8 characters long, 
slightly differently. Calling pg_xlogfile_name seemed a bit indirect.

I expanded the documentation on how to download and install IPC::Run. I 
instructed to add PERL5LIB to buildenv.pl, pointing it to the extracted 
IPC-Run-0.94/lib directory. Your phrasing of "putting it into PERL5LIB" 
was pretty vague, and in fact the PERL5LIB environment variable was 
overwritten in vcregress.pl, so just setting the PERL5LIB variable 
wouldn't have worked. I changed vcregress.pl to not do that.

I considered moving the instructions on downloading and installing 
IPC::Run in the Requirements section, instead of the Running the 
Regression Tests section. But there are additional instructions on 
downloading and installing more dependencies in the Building the 
Documentation section too. The other dependencies in the Requirements 
section affect the built binaries. Well, except for the Diff utility. 
We're not totally consistent, but I think it's OK as it is.

> Also I added an equivalent of --enable-tap-tests for this MSVC stuff,
> removed afterwards by Heikki. Do we want it or not?

On Unix, that option controls whether "make check-world" runs the TAP 
tests, but there is no equivalent of "check-world" on Windows. Maybe 
there should be, but as long as there isn't, the only thing that the 
option did was to stop "vcregress tapcheck" from working, if didn't 
enable it in the config file first. That seems silly, so I removed it. 
We'll need it if we add an equivalent of "make check-world" to 
vcregress, but let's cross that bridge when we get there.

- Heikki


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers