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

From: Rahila Syed <rahilasyed90(at)gmail(dot)com>
To: Matheus Alcantara <matheusssilv97(at)gmail(dot)com>
Cc: 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-02 19:56:34
Message-ID: CAH2L28uoo98D+u3dY319Ja4SWAV=mt19MkruWJjp7=N-Sqbn4g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

Hi,

Thank you both for the patch and the explanation regarding the fix.

> Another approach would be to change the
> signature of expand_dynamic_library_name() to receive a boolean and only
> srip the $libdir if it's true, but I think that the expand function
> should not handle the strip logic, it should only expand the dynamic
> library path and nothing else.
>

The reasoning makes sense to me.

> I'm also attaching a second patch that has a TAP test case that
> reproduce this issue, it has some hacks to move dynamic library files
> around to exercice the problem so I'm not sure if it could be good to
> also include these tests on the final patch.
>

The TAP test is useful in testing various paths of extension files.
It will be cleaner if the dummy_index_am.so is deleted at the end of the
tests from the $libdir.

Currently, the tests seem to be failing as follows for windows:

[19:07:13.107] # Failed test 'create extension from custom extension
control path'[19:07:13.107] # at
C:/cirrus/src/test/modules/test_extensions/t/002_strip_modulepathname.pl
line 102.[19:07:13.107] # got: '2'[19:07:13.107] #
expected: '0'[19:07:13.107] # Looks like you failed 1 test of
7.[19:07:13.107] [19:07:13.107] (test program exited with status code
1)

Details here: Windows - Server 2019, VS 2019 - Meson & ninja - Cirrus CI
<https://cirrus-ci.com/task/5241991606304768>

Thank you,
Rahila Syed

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Matheus Alcantara 2025-06-02 20:53:23 Re: BUG #18920: LOAD '$libdir/plugins' no longer works in 18beta1
Previous Message Masahiko Sawada 2025-06-02 18:08:54 Re: Logical replication 'invalid memory alloc request size 1585837200' after upgrading to 17.5