Re: Strip -mmacosx-version-min options from plperl build

From: Andres Freund <andres(at)anarazel(dot)de>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Strip -mmacosx-version-min options from plperl build
Date: 2022-08-26 14:14:32
Message-ID: 20220826141432.kqvir4depkndlx7c@awork3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2022-08-26 10:04:35 -0400, Andrew Dunstan wrote:
> On 2022-08-25 Th 18:13, Andres Freund wrote:
> >>> Could you show objdump -t of the library? Perhaps once with the flags as now,
> >>> and once relinking with the "old" flags that we're now omitting?
> >>
> >> current:
> >>
> >>
> >> $ objdump -t plperl.dll | egrep '_PG_init|Pg_magic_func'
> >> [103](sec  1)(fl 0x00)(ty  20)(scl   2) (nx 0) 0x00000000000040a0
> >> Pg_magic_func
> >> [105](sec  1)(fl 0x00)(ty  20)(scl   2) (nx 0) 0x00000000000040b0 _PG_init
> >>
> >>
> >> from July 11th build:
> >>
> >>
> >> $ objdump -t plperl.dll | egrep '_PG_init|Pg_magic_func'
> >> [101](sec  1)(fl 0x00)(ty  20)(scl   2) (nx 0) 0x00000000000040d0
> >> Pg_magic_func
> >> [103](sec  1)(fl 0x00)(ty  20)(scl   2) (nx 0) 0x00000000000040e0 _PG_init
> > Thanks.
> >
> > So it looks like it's not the symbol not being exported. I wonder if the image
> > base thing is somehow the problem? Sounds like it should just be an efficiency
> > difference, by avoiding some relocations, not a functional difference.
> >
> > Can you try adding just that to the flags for building and whether that then
> > allows a LOAD 'plperl' to succeed?
> >
>
>
> Adding what?

-Wl,--enable-auto-image-base

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2022-08-26 14:23:41 Re: standby promotion can create unreadable WAL
Previous Message Robert Haas 2022-08-26 14:11:38 Re: has_privs_of_role vs. is_member_of_role, redux