Re: pgsql: Add pg_alterckey utility to change the cluster key - Mailing list pgsql-committers

From Michael Paquier
Subject Re: pgsql: Add pg_alterckey utility to change the cluster key
Date
Msg-id X+b/dR6P4tPpm6kt@paquier.xyz
Whole thread Raw
In response to Re: pgsql: Add pg_alterckey utility to change the cluster key  (Pavel Stehule <pavel.stehule@gmail.com>)
Responses Re: pgsql: Add pg_alterckey utility to change the cluster key
List pgsql-committers
On Sat, Dec 26, 2020 at 08:29:10AM +0100, Pavel Stehule wrote:
> I did recheck with same result

The Makefile of pg_alterckey is busted, and adding --enable-tap-tests
to the options of ./configure is enough to see a failure.  In short,
src/bin/pg_alterckey/Makefile includes the following lines, but it has
no TAP tests as of pg_alterckey/t/:
check:
        $(prove_check)

installcheck:
        $(prove_installcheck)

So this breaks.

I would like to point out that all non-Unix buildfarm members are
broken like fairywen because of the addition of those scripts:
https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=fairywren&dt=2020-12-26%2009%3A04%3A27
/usr/bin/install: cannot stat 'crypto/ckey_aws.sh.sample': No such
file or directory

The CF bot at http://cfbot.cputube.org/ includes tests on Windows, so
those problems would have been detected beforehand.  Did you look at
these?  If this cannot be fixed, could it be possible to revert
please?  It looks rather clear that this has not been tested across
multiple platforms, and the absence of tests to allow the buildfarm to
stress this code does not really help either in gaining confidence
that this is stable.
--
Michael

Attachment

pgsql-committers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: pgsql: Add pg_alterckey utility to change the cluster key
Next
From: Michael Paquier
Date:
Subject: Re: pgsql: Add pg_alterckey utility to change the cluster key