diff mbox series

[FFmpeg-devel,v2] configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm

Message ID ZYiHi48M9JH99JMZ@humpty.home.comstyle.com
State New
Headers show
Series [FFmpeg-devel,v2] configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Brad Smith Dec. 24, 2023, 7:33 p.m. UTC
configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm

Fixes the build. It's a requirement when utilizing PIE.

Signed-off-by: Brad Smith <brad@comstyle.com>
---
 configure | 2 ++
 1 file changed, 2 insertions(+)

Comments

Michael Niedermayer Dec. 25, 2023, 2:08 a.m. UTC | #1
On Sun, Dec 24, 2023 at 02:33:31PM -0500, Brad Smith wrote:
> configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm
> 
> Fixes the build. It's a requirement when utilizing PIE.
> 
> Signed-off-by: Brad Smith <brad@comstyle.com>
> ---
>  configure | 2 ++
>  1 file changed, 2 insertions(+)

will apply

thx

[...]
Brad Smith Dec. 26, 2023, 2:04 a.m. UTC | #2
On 2023-12-24 9:08 p.m., Michael Niedermayer wrote:
> On Sun, Dec 24, 2023 at 02:33:31PM -0500, Brad Smith wrote:
>> configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm
>>
>> Fixes the build. It's a requirement when utilizing PIE.
>>
>> Signed-off-by: Brad Smith <brad@comstyle.com>
>> ---
>>   configure | 2 ++
>>   1 file changed, 2 insertions(+)
> will apply
>
> thx

Thanks. Can you please back port this to 6.1, 6.0, 5.1, 5.0 and 4.4?
Michael Niedermayer Dec. 26, 2023, 12:34 p.m. UTC | #3
On Mon, Dec 25, 2023 at 09:04:54PM -0500, Brad Smith wrote:
> On 2023-12-24 9:08 p.m., Michael Niedermayer wrote:
> > On Sun, Dec 24, 2023 at 02:33:31PM -0500, Brad Smith wrote:
> > > configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm
> > > 
> > > Fixes the build. It's a requirement when utilizing PIE.
> > > 
> > > Signed-off-by: Brad Smith <brad@comstyle.com>
> > > ---
> > >   configure | 2 ++
> > >   1 file changed, 2 insertions(+)
> > will apply
> > 
> > thx
> 
> Thanks. Can you please back port this to 6.1, 6.0, 5.1, 5.0 and 4.4?

it will be in the next releases from these branches

thx

[...]
Brad Smith Dec. 27, 2023, 3:33 a.m. UTC | #4
On December 26, 2023 7:35:09 a.m. Michael Niedermayer 
<michael@niedermayer.cc> wrote:

> On Mon, Dec 25, 2023 at 09:04:54PM -0500, Brad Smith wrote:
>> On 2023-12-24 9:08 p.m., Michael Niedermayer wrote:
>>> On Sun, Dec 24, 2023 at 02:33:31PM -0500, Brad Smith wrote:
>>>> configure: Enable section_data_rel_ro for FreeBSD and NetBSD aarch64 / arm
>>>>
>>>> Fixes the build. It's a requirement when utilizing PIE.
>>>>
>>>> Signed-off-by: Brad Smith <brad@comstyle.com>
>>>> ---
>>>> configure | 2 ++
>>>> 1 file changed, 2 insertions(+)
>>> will apply
>>>
>>> thx
>>
>> Thanks. Can you please back port this to 6.1, 6.0, 5.1, 5.0 and 4.4?
>
> it will be in the next releases from these branches
>
> thx

Thanks.


Sent with Aqua Mail for Android
diff mbox series

Patch

diff --git a/configure b/configure
index 078aa9437a..226874600a 100755
--- a/configure
+++ b/configure
@@ -5601,6 +5601,7 @@  case $target_os in
         ;;
     netbsd)
         disable symver
+        enable section_data_rel_ro
         oss_indev_extralibs="-lossaudio"
         oss_outdev_extralibs="-lossaudio"
         enabled gcc || check_ldflags -Wl,-zmuldefs
@@ -5619,6 +5620,7 @@  case $target_os in
         disable symver
         ;;
     freebsd)
+        enable section_data_rel_ro
         ;;
     bsd/os)
         add_extralibs -lpoll -lgnugetopt