Re: Re[2]: bool_plperl transform

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: Re[2]: bool_plperl transform
Дата
Msg-id 5578.1583097237@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re[2]: bool_plperl transform  (Wao <wao@mail.ru>)
Ответы Re[4]: bool_plperl transform  (Ivan Panchenko <wao@mail.ru>)
Список pgsql-hackers
=?UTF-8?B?V2Fv?= <wao@mail.ru> writes:
> Please find the full patch attached.

The cfbot shows this failing to build on Windows:

https://ci.appveyor.com/project/postgresql-cfbot/postgresql/build/1.0.81889

I believe that's a build without plperl, so what it's probably telling
you is that Mkvcbuild.pm needs to be taught to build this module
conditionally, as it already does for hstore_plperl and jsonb_plperl.

Also, while the Linux build is passing, I can't find that it is actually
compiling or testing bool_plperl anywhere:

https://travis-ci.org/postgresql-cfbot/postgresql/builds/656909114

This is likely because you didn't add it to contrib/Makefile.

In general, I'd suggest grepping for references to hstore_plperl
or jsonb_plperl, and making sure that bool_plperl gets added where
appropriate.

I rather imagine you need a .gitignore file, as well.

You're also going to have to provide some documentation, because
I don't see any in the patch.

            regards, tom lane



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

Предыдущее
От: David Steele
Дата:
Сообщение: Commitfest 2020-03 Now in Progress
Следующее
От: Fabien COELHO
Дата:
Сообщение: Re: pgbench: option delaying queries till connectionsestablishment?