Web lists-archives.com

Re: [MPlayer-dev-eng] [PATCH 2/2] Fix compilation with recent FFmpeg on ARM




Hi Laurent!

On 2015-03-13 18:18 -0400, Laurent wrote:
> Not sure about this one: HAVE_SECTION_DATA_REL_RO.
> It is defined for Android and Linux, see:
> http://lists.libav.org/pipermail/libav-commits/2014-December/015860.html
> So I lazily hardcoded it:

  Thank you for both of your patches. Fixes build on ARM here.

  Your patch got line-wrapped though. You should be able to
notice it by e.g. by looking at the ML archive. If you cannot
get it through unmangled, consider attaching the patch next time.
For this time I repaired it manually myself.

  As for the question for when to define HAVE_SECTION_DATA_REL_RO:
I do not have much knowledge about the ARM platform. I only used
it with Linux so far. As far as Linux vs Android is concerned, I
do not think we differentiate Linux and Android in MPlayer at all.

  Maybe some other developer can comment. Reimar?

  Alexander

> diff --git a/configure b/configure
> index de7c6e4..77115ed 100755
> --- a/configure
> +++ b/configure
> @@ -9363,6 +9363,7 @@ $def_gnutls
>  #define HAVE_MM_EMPTY 0
>  #define HAVE_PPC4XX 0
>  #define HAVE_RDTSC 0
> +#define HAVE_SECTION_DATA_REL_RO 1
>  #define HAVE_SIMD_ALIGN_16 1
>  #define HAVE_STRERROR_R 0
>  #define HAVE_STRPTIME 0

Attachment: pgprJA6apTyuz.pgp
Description: PGP signature

_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng