[MPlayer-dev-eng] [PATCH 2/2] Fix compilation with recent FFmpeg on ARM
eclipse7 at gmx.net
Sun Mar 15 02:21:52 CET 2015
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:
> 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?
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 181 bytes
Desc: not available
More information about the MPlayer-dev-eng