Re: pgsql: Add TAP tests for pg_dump

Поиск
Список
Период
Сортировка
От Stephen Frost
Тема Re: pgsql: Add TAP tests for pg_dump
Дата
Msg-id 20160506193957.GX10850@tamriel.snowman.net
обсуждение исходный текст
Ответ на Re: pgsql: Add TAP tests for pg_dump  (Stephen Frost <sfrost@snowman.net>)
Ответы Re: pgsql: Add TAP tests for pg_dump  (Stephen Frost <sfrost@snowman.net>)
Re: pgsql: Add TAP tests for pg_dump  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-committers
* Stephen Frost (sfrost@snowman.net) wrote:
> * Stephen Frost (sfrost@snowman.net) wrote:
> > src/test/modules/test_pg_dump/Makefile             |   25 +
> > src/test/modules/test_pg_dump/README               |    2 +
> > .../modules/test_pg_dump/expected/test_pg_dump.out |    6 +
> > src/test/modules/test_pg_dump/sql/test_pg_dump.sql |    1 +
> > src/test/modules/test_pg_dump/t/001_base.pl        |  535 ++++
> > .../modules/test_pg_dump/test_pg_dump--1.0.sql     |   15 +
> > src/test/modules/test_pg_dump/test_pg_dump.control |    3 +
>
> Looks like the test_pg_dump extension made the Windows builds upset.
> I'm guessing that's because I set 'MODULES_big' even though there isn't
> a .c component.
>
> Doing a local build with that commented out, assuming that works and
> doesn't generate the .so any more on my Linux box, I'll push the change
> up to hopefully fix those buildfarm members.

Alright, apparently that made other Windows buildfarm members unhappy...

I guess the next approach will be to add back MODULES_big and add in a
.c file for the Windows systems to be happy about.  I'm certainly open
to other suggestions.

The intagg contrib modules doesn't have a .c file either, nor a
MODULES_big line, and I don't see any errors with it.  I'm not terribly
familiar with how the Windows builds are run though.

Thanks!

Stephen

Вложения

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

Предыдущее
От: Stephen Frost
Дата:
Сообщение: pgsql: Remove MODULES_big from test_pg_dump
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: pgsql: Add TAP tests for pg_dump