From: | Alexander Lakhin <exclusion(at)gmail(dot)com> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: make installcheck-world in a clean environment |
Date: | 2018-04-06 06:19:01 |
Message-ID: | 91833017-8f78-a51e-f229-55ad467ffef3@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
02.04.2018 12:12, Alexander Lakhin wrote:
>
> Is it a supported scenario to make installcheck-world without
> performing "make" first?
> (If I do "make -C src/interfaces/ecpg" and then "make
> installcheck-world", then this error is gone. And when I set up all
> the extensions, all tests passed successfully.)
> And even if we need to perform make, I wonder, should the recompiled
> ecpg binary be checked instead of installed one?
> I tried to modify Makefile to target installed ecpg binary and it's
> libs (see the patch attached), it works, but this fix is only suited
> for installcheck.
> So if this scenario should be supported, a more elaborated fix is needed.
To avoid overheating of this pretty hot discussion, I would like just to
propose "a more elaborated fix" (for REL_10_STABLE and master).
In fact, when we perform "make installcheck" it not only requires us to
build ecpg, but it also rebuilds libpostgres, libpgport and libpq (for
installcheck-world).
I believe that the larger testing surface (coverage), the better, so
using installed assets (libs, headers) is more useful.
Regarding "remote installcheck", that was discussed recently, the
proposed patch complicates this, but opens a way to implement it correctly.
Think of distinct target "remotecheck", that will not define
USE_INSTALLED_ASSETS, but will account for remote connection to server
and run only supported tests.
Best regards,
------
Alexander Lakhin
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company
Attachment | Content-Type | Size |
---|---|---|
make-checkinstall-10.diff | text/x-patch | 6.5 KB |
make-checkinstall-master.diff | text/x-patch | 6.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2018-04-06 06:21:12 | Re: Optimizing nested ConvertRowtypeExpr execution |
Previous Message | Pavel Stehule | 2018-04-06 06:14:19 | Re: some last patches breaks plan cache |