Re: BUG #18920: LOAD '$libdir/plugins' no longer works in 18beta1

From: Matheus Alcantara <matheusssilv97(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Rahila Syed <rahilasyed90(at)gmail(dot)com>, evsi(at)amazon(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #18920: LOAD '$libdir/plugins' no longer works in 18beta1
Date: 2025-06-03 12:31:17
Message-ID: CAFY6G8ezmtNXV-VOeZ234dRz5agwwU-UMo_2PgdNUz3AVY9H6g@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On 03/06/25 02:48, Michael Paquier wrote:
> I don't like much the tests you are adding here. First, the
> cross-platform requirements to copy a library to the plugins/ folder
> is annoying. The actual issue is that we don't have an installation
> rule to be able to install a library to a plugins/ path to allow
> non-superusers to load it, no? I mean, as 4f7f7b037585 states, we
> have a `make install prefix=/else/where` but the TAP tests cannot
> shape a temporary installation with it. If we had something like that
> for meson and makefiles, we could then reuse EXTRA_INSTALL to force a
> library to exist where we want it to be, for the sake of testing
> coverage.
>
> I am not completely sure that the tests are completely waterproof,
> either. Some distributions like fancy installation folder layers,
> like Debian, and such things have proven to break these folk's tests.
> Having a centralized rule could be also useful for out-of-core
> extensions, to give these a way to install something inside a plugins/
> folder. At least that may be better than requiring pg_config to get
> the basic library install path.
>
> Second, requiring dummy_index_am inside the tests test_extensions is
> adding unwelcome complexity across the test modules.
>

I also don't like these tests, it has a lot of hacks as I've mentioned in
my first email and I'm almost sure that if we push this it will not work
on a lot of build farm animals. I just wrote these tests to make it
possible to reproduce the issue on my local machine and make the fix
easier, and I just shared here so that folks can also have a way to
reproduce the issue and maybe share other ideas to test this.

--
Matheus Alcantara

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Sandeep Thakkar 2025-06-03 13:30:02 Re: BUG #18946: Installation Setup File
Previous Message vignesh C 2025-06-03 12:24:16 Re: Logical replication 'invalid memory alloc request size 1585837200' after upgrading to 17.5