Re: pgsql: Add pg_alterckey utility to change the cluster key

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: pgsql: Add pg_alterckey utility to change the cluster key
Дата
Msg-id X+b/dR6P4tPpm6kt@paquier.xyz
обсуждение исходный текст
Ответ на Re: pgsql: Add pg_alterckey utility to change the cluster key  (Pavel Stehule <pavel.stehule@gmail.com>)
Ответы Re: pgsql: Add pg_alterckey utility to change the cluster key
Список 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

Вложения

В списке pgsql-committers по дате отправления:

Предыдущее
От: Pavel Stehule
Дата:
Сообщение: Re: pgsql: Add pg_alterckey utility to change the cluster key
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: pgsql: Add pg_alterckey utility to change the cluster key