| From: | "matsumura(dot)ryo(at)fujitsu(dot)com" <matsumura(dot)ryo(at)fujitsu(dot)com> |
|---|---|
| To: | 'Michael Paquier' <michael(at)paquier(dot)xyz> |
| Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | RE: [bugfix]"make installcheck" could not work in PGXS |
| Date: | 2020-08-07 01:43:49 |
| Message-ID: | OSAPR01MB50272ECB3A14492B92AC1445E8490@OSAPR01MB5027.jpnprd01.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello,
On Fri, Aug 5, 2020 at 10:57:56 +0000, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> Yeah, that's a known problem. One way to counter that is for example
> to grab the path of pg_regress from pg_config --libdir and set
> $ENV{PG_REGRESS} to it, but that's hacky. So I agree that it would be
> good to do something.
Thank you.
I attach a new patch.
> Not sure that this goes completely to the right direction. It seems
> to me that we should have room to set and use PG_REGRESS also for
> pg_regress_check and pg_regress_installcheck.
I understand that PG_REGRESS is an environment variable for each test program.
So I add a gmake variable PG_REGRESS_PATH.
The followings are other changings.
- Change REGRESS_SHLIB like as PG_REGRESS.
- Replace $(CURDIR)/$(top_builddir) to $(abs_top_builddir).
- Remove setting of environment variable 'top_builddir' in command line for prove_installcheck.
I wonder what I should set to it and then I remove it.
Because top_builddir is used for gmake genellaly not for test programs and PostgreSQL's test framework doesn't use it.
Is it going too far?
Regards
Ryo Matsumura
| Attachment | Content-Type | Size |
|---|---|---|
| bug_pgxs_installcheck.v1.1.patch | application/octet-stream | 2.2 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit Kapila | 2020-08-07 01:48:55 | Re: display offset along with block number in vacuum errors |
| Previous Message | Tom Lane | 2020-08-07 01:08:40 | Re: Should the nbtree page split REDO routine's locking work more like the locking on the primary? |