From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Peter Eisentraut <peter_e(at)gmx(dot)net>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: no test programs in contrib |
Date: | 2014-11-27 20:10:04 |
Message-ID: | 11133.1417119004@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
> I have also changed things so that:
> 1. test modules are not installed by "make install", not checked by
> "make installcheck", not checked by "make check".
> 2. test modules are checked by "make check-world" (this is consistent
> with handling of contrib).
> 3. test modules are checked by "make installcheck-world" (this is
> consistent with handling of contrib)
> 4. test modules are installed by "make install-world". This is
> consistent with contrib, and it's necessary so that "make
> installcheck-world" passes.
I'm not too happy with that approach, because packagers are going to
tend to think they should package any files installed by install-world.
The entire point of this change, IMO, is that the test modules should
*not* get installed, certainly not by normal install targets. Being
consistent with the existing contrib packaging is exactly not what we
want.
Maybe we should only allow check-world to run these tests, and not
installcheck-world? That's kind of annoying, but what you are
doing now seems to defeat the purpose altogether.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2014-11-27 20:30:04 | Re: [REVIEW] Re: Compression of full-page-writes |
Previous Message | Alvaro Herrera | 2014-11-27 19:24:20 | Re: no test programs in contrib |