Re: tap tests remove working directories

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: tap tests remove working directories
Date: 2015-08-09 12:41:38
Message-ID: CAB7nPqRjqRanu1JoNj3rhAEWNn_cG5aB2==3sZTG5bL8Yz3B7g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Aug 9, 2015 at 1:40 AM, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote:
>
> On 08/08/2015 09:31 AM, Robert Haas wrote:
>>
>> On Fri, Aug 7, 2015 at 7:17 PM, Andrew Dunstan <andrew(at)dunslane(dot)net>
>> wrote:
>>>
>>> That certainly isn't what happens, and given the way this is done in
>>> TestLib.pm, using the CLEANUP parameter of File::Temp's tempdir()
>>> function,
>>> it's not clear how we could do that easily.
>>
>> <shot-in-the-dark>
>>
>> Set cleanup to false and manually remove the directory later in the
>> code, so that stuff runs only if we haven't died sooner?
>>
>> </shot-in-the-dark>
>>
> Yeah, maybe. I'm thinking of trying to do it more globally, like in
> src/Makefile.global.in. That way we wouldn't have to add new code to every
> test file.

If we rely on END to clean up the temporary data folder, there is no
need to impact each test file, just the test functions called in
TestLib.pm that could switch a flag to not perform any cleanup at the
end of the run should an unexpected result be found. Now I am as well
curious to see what you have in mind with manipulating
Makefile.global.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andrew Dunstan 2015-08-09 14:19:45 Re: tap tests remove working directories
Previous Message Michael Paquier 2015-08-09 11:56:06 Re: make check-world problem