Re: Disabling src/test/[ssl|ldap] when not building with SSL/LDAPsupport

On 3/2/18 03:23, Michael Paquier wrote:
> That's a better idea.  I have reworked that in 0001.  What do you think?
> This has the same effect as diag(), which shows information directly to
> the screen, and that's the behavior I was looking for.

I ended up using plan skip_all, which seems to be intended for this
purpose, according to the documentation.

> I have concluded about using the most simple name: PG_TEST_EXTRA.  A
> documentation attempt is added as well.  This is unfortunately close to
> EXTRA_TESTS.  It would be tempting to directly reuse EXTRA_TESTS as
> well, however this is used only for the core regression tests now, so a
> separated variable seems adapted to me.

Committed.  Very nice.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services