<div dir='auto'><div>Hi,</div><div dir="auto"><br></div><div dir="auto">Thank you for the details,</div><div dir="auto"><br></div><div dir="auto">If this is a dolt issue, will it be fixed with the meson buildsystem ?</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">-- </div><div dir="auto">Alexandre Janniaux</div><div dir="auto">Videolabs</div><div dir="auto"><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">Le 15 oct. 2019 22:46, Marvin Scholz <epirat07@gmail.com> a écrit :<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">On 15 Oct 2019, at 17:16, Rémi Denis-Courmont wrote:</p>
<p dir="ltr">> Hi,<br>
><br>
> Text relocations don't work on Linux and ELF platforms in general, <br>
> AFAIK. So this needs proper fixing if it's *really* text relocations.</p>
<p dir="ltr">I’ve investigate this issue more.<br>
I turns out it is only an issue when configuring libvlc with <br>
--enable-static --disable-shared<br>
which causes libtool to not default to PIC, as expected. But even when <br>
using --with-pic<br>
it does not use PIC which in fact isn’t libtools fault but DOLT, which <br>
does not handle<br>
this correctly.</p>
<p dir="ltr">I’ve submitted another patch that reverts this one and adds a <br>
workaround for that.<br>
It’s not a perfect fix, just a workaround because dolt.m4 is quite <br>
hard to read and follow,<br>
so I really can’t figure out how to patch that to do the right thing.</p>
<p dir="ltr">Just opting out of DOLT in case of --with-pic being given does not work <br>
either, because<br>
the --tag=ASM flag used for NASM compilation with libtool is only <br>
supported by the DOLT<br>
wrapper script, not by the real libtool…</p>
<p dir="ltr">><br>
> Le 15 octobre 2019 15:55:31 GMT+03:00, Marvin Scholz <br>
> <epirat07@gmail.com> a écrit :<br>
>> It contains text relocations which are not allowed on Android.<br>
>> ---<br>
>> modules/video_filter/Makefile.am              | 3 +++<br>
>> modules/video_filter/deinterlace/algo_yadif.c | 2 +-<br>
>> 2 files changed, 4 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/modules/video_filter/Makefile.am<br>
>> b/modules/video_filter/Makefile.am<br>
>> index 51ee92aff1..6f1714c683 100644<br>
>> --- a/modules/video_filter/Makefile.am<br>
>> +++ b/modules/video_filter/Makefile.am<br>
>> @@ -136,9 +136,12 @@ libdeinterlace_plugin_la_SOURCES = \<br>
>>        video_filter/deinterlace/algo_ivtc.c<br>
>> video_filter/deinterlace/algo_ivtc.h<br>
>> # inline ASM doesn't build with -O0<br>
>> libdeinterlace_plugin_la_CFLAGS = $(AM_CFLAGS) -O2<br>
>> +# Disabled on Android due to text reloc issues<br>
>> +if !HAVE_ANDROID<br>
>> if HAVE_X86ASM<br>
>> libdeinterlace_plugin_la_SOURCES +=<br>
>> video_filter/deinterlace/yadif_x86.asm<br>
>> endif<br>
>> +endif<br>
>> if HAVE_NEON<br>
>> libdeinterlace_plugin_la_SOURCES +=<br>
>> video_filter/deinterlace/merge_arm.S<br>
>> libdeinterlace_plugin_la_CFLAGS += -DCAN_COMPILE_ARM<br>
>> diff --git a/modules/video_filter/deinterlace/algo_yadif.c<br>
>> b/modules/video_filter/deinterlace/algo_yadif.c<br>
>> index 6fdecf53a5..98dbbe9b04 100644<br>
>> --- a/modules/video_filter/deinterlace/algo_yadif.c<br>
>> +++ b/modules/video_filter/deinterlace/algo_yadif.c<br>
>> @@ -112,7 +112,7 @@ int RenderYadif( filter_t *p_filter, picture_t<br>
>> *p_dst, picture_t *p_src,<br>
>> void (*filter)(uint8_t *dst, uint8_t *prev, uint8_t *cur, uint8_t<br>
>> *next,<br>
>>                    int w, int prefs, int mrefs, int parity, int <br>
>> mode);<br>
>><br>
>> -#if defined(HAVE_X86ASM)<br>
>> +#if defined(HAVE_X86ASM) && !defined (__ANDROID__)<br>
>>         if( vlc_CPU_SSSE3() )<br>
>>             filter = vlcpriv_yadif_filter_line_ssse3;<br>
>>         else<br>
>> -- <br>
>> 2.20.1 (Apple Git-117)<br>
>><br>
>> _______________________________________________<br>
>> vlc-devel mailing list<br>
>> To unsubscribe or modify your subscription options:<br>
>> https://mailman.videolan.org/listinfo/vlc-devel<br>
><br>
> -- <br>
> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez <br>
> excuser ma brièveté._______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> https://mailman.videolan.org/listinfo/vlc-devel<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
https://mailman.videolan.org/listinfo/vlc-devel</p>
</blockquote></div><br></div></div></div>