|From:||ilmari(at)ilmari(dot)org (Dagfinn Ilmari =?utf-8?Q?Manns=C3=A5ker?=)|
|To:||Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>|
|Cc:||Craig Ringer <craig(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>|
|Subject:||Re: configure's checks for --enable-tap-tests are insufficient|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
> ilmari(at)ilmari(dot)org (Dagfinn Ilmari =?utf-8?Q?Manns=C3=A5ker?=) writes:
>> Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
>>> ilmari(at)ilmari(dot)org (Dagfinn Ilmari =?utf-8?Q?Manns=C3=A5ker?=) writes:
>>>> I think the other way would be better simpler: we require a complete
>>>> Perl core installation.
>>> And you propose to test for that in configure how?
>> Good point, I'd overlooked that aspect. In that case we can't rely on
>> what Debian or others happen to include in their minimal packages, we
>> have to explicitly check for _every_ module we use anywhere in the test
>> suite, core or not (except perhaps pragmas like 'strict' and 'warnings').
> That seems like expensive overkill to me, especially since it seems
> unlikely that we'd keep the list up-to-date over time.
The patches uptread add about 0.2s to a 6.9s configure run here, I don't
consider that very expensive. We don't add new modules very often, so I
don't think it's much of a maintenance overhead either.
> I just want to test for modules that we know are likely to be omitted
> on popular platforms. I've proposed testing two that would improve
> the user experience on Red Hat; what's the equivalent minimum set for
On Debian (I've checked the previous and current stable releases, as
well as unstable) the only package required by the TAP tests not in
perl-base is Time::HiRes. Of the modules required by PL/Perl, Opcode is
not in perl-base either.
BTW, should include the version in the Test::More check, since we only
requie Perl 5.8, but that only shipped Test::More 0.47, while we require
"A disappointingly low fraction of the human race is,
at any given time, on fire." - Stig Sandbeck Mathisen
|Next Message||Tom Lane||2018-03-20 17:27:35||Re: pg_get_functiondef forgets about most GUC_LIST_INPUT GUCs|
|Previous Message||Garima Natani||2018-03-20 17:06:05||GSOC 2018 Proposal review|