Re: [PATCH] Allow TAP tests to be run individually

On 14 November 2016 at 16:52, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
> On Mon, Nov 14, 2016 at 3:45 PM, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>> On 11 November 2016 at 18:13, Michael Paquier <michael(dot)paquier(at)gmail(dot)com> wrote:
>>> On Fri, Nov 11, 2016 at 6:10 PM, Craig Ringer <craig(at)2ndquadrant(dot)com> wrote:
>>>> Please backpatch to at least 9.6 since it's trivial and we seem to be
>>>> doing that for TAP. 9.5 and 9.4 would be nice too :)
>>>
>>> Yes please!
>>
>> No immediate takers, so adding to CF.
>>
>> I've taken the liberty of adding you as a reviewer based on your
>> response and the simplicity of the patch. if you get the chance to
>> test and verify please set ready for committer.
>
> I don't mind. This patch uses the following pattern:
> $(or $(PROVE_TESTS),t/*.pl)
> While something more spread in Postgres source would be something like that:
> $(if $(PROVE_TESTS),$(PROVE_TESTS),t/*.pl)
> It seems to me that we'd prefer that for consistency, but I see no
> reason to not keep your patch as well. I am marking that as ready for
> committer.

Thanks.

--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services