Re: Equivalent of --enable-tap-tests in MSVC scripts
Andrew Dunstan <andrew@dunslane.net>
From: Andrew Dunstan <andrew@dunslane.net>
To: Michael Paquier <michael.paquier@gmail.com>,
PostgreSQL mailing lists <pgsql-hackers@postgresql.org>
Date: 2016-03-01T15:40:12Z
Lists: pgsql-hackers
On 03/01/2016 08:00 AM, Michael Paquier wrote:
> Hi all,
>
> As of now the MSVC scripts control if TAP tests are enabled or not
> using a boolean flag as $config->{tap_tests}. However, this flag is
> just taken into account in vcregress.pl, with the following issues:
> 1) config_default.pl does not list tap_tests, so it is unclear to
> users to enable them. People need to look into vcregress.pl as a start
> point.
> 2) GetFakeConfigure() does not translate $config->{tap_tests} into
> --enable-tap-tests, leading to pg_config not reporting it in
> CONFIGURE. This is inconsistent with what is done in ./configure.
>
> Attached is a patch to address those two issues.
>
Good work. There seem to be some unrelated whitespace changes. Shouldn't
this just be two extra lines?
cheers
andrew
Commits
-
Add 'tap_tests' flag in config_default.pl
- c7327561f757 9.4.7 landed
- 9b3f0c10e7ae 9.5.2 landed
- 52fe6f4e02f8 9.6.0 landed