From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Noah Misch <noah(at)leadboat(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: PXGS vs TAP tests |
Date: | 2021-07-01 13:13:25 |
Message-ID: | 53a6dc1b-f4d8-bec2-d4c4-6ded758f25f9@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 6/21/21 8:23 PM, Michael Paquier wrote:
> On Sun, Jun 20, 2021 at 01:24:04PM -0400, Andrew Dunstan wrote:
>> Tests pass with the attached patch, which puts the setting in the
>> Makefile for the recovery tests. The script itself doesn't need any
>> changing.
> +REGRESS_SHLIB=$(abs_top_builddir)/src/test/regress/regress$(DLSUFFIX)
> +export REGRESS_SHLIB
> It may be better to add a comment here explaning why REGRESS_SHLIB is
> required in this Makefile then?
>
> While on it, could we split those commands into multiple lines and
> reduce the noise of future diffs? Something as simple as that would
> make those prove commands easier to follow:
> +cd $(srcdir) && TESTDIR='$(CURDIR)' \
> + $(with_temp_install) \
> + PGPORT='6$(DEF_PGPORT)' \
> + PG_REGRESS='$(CURDIR)/$(top_builddir)/src/test/regress/pg_regress' \
> + REGRESS_SHLIB= '$(abs_top_builddir)/src/test/regress/regress$(DLSUFFIX)' \
> + $(PROVE) $(PG_PROVE_FLAGS) $(PROVE_FLAGS) $(if $(PROVE_TESTS),$(PROVE_TESTS),t/*.pl)
>
> There are other places where this could happen, but the TAP commands
> are particularly long.
OK, done.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dean Rasheed | 2021-07-01 13:17:46 | Re: Numeric x^y for negative x |
Previous Message | Amit Kapila | 2021-07-01 13:02:06 | Re: Refactor "mutually exclusive options" error reporting code in parse_subscription_options |