[FFmpeg-devel] [FFmpeg-commits] configure: remove early check_deps $ARCH_EXT_LIST

[Martin Storsjö]

On Fri, 11 Feb 2011, Mans Rullgard  wrote:

> Module: ffmpeg
> Branch: master
> Commit: 4b884207eb7319577f8ceb94a56f42c5ac0ab294
> 
> Author: Mans Rullgard <mans at mansr.com>
> Date:   Fri Feb 11 18:28:12 2011 +0000
> 
> configure: remove early check_deps $ARCH_EXT_LIST
> 
> The early disabling of irrelevant arch extensions is no longer
> required, and removing it makes dependencies involving these
> work as expected.
> 
> Signed-off-by: Mans Rullgard <mans at mansr.com>
> 
> ---
> 
>  configure |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/configure b/configure
> index a3ca15d..a310d25 100755
> --- a/configure
> +++ b/configure
> @@ -2494,8 +2494,6 @@ die_license_disabled version3 libopencore_amrwb
>  
>  enabled version3 && { enabled gpl && enable gplv3 || enable lgplv3; }
>  
> -check_deps $ARCH_EXT_LIST
> -
>  disabled optimizations || check_cflags -fomit-frame-pointer
>  
>  enable_pic() {

This change broke compilation for Windows CE, where configure now errors 
out with this error:

Error, no aligned memory allocator but SSE enabled, disable it or use 
--enable-memalign-hack.

This is checked at this point:

if ! enabled_any memalign memalign_hack posix_memalign malloc_aligned &&
     enabled_any $need_memalign ; then
    die "Error, no aligned memory allocator but SSE enabled, disable it or use --enable-memalign-hack."
fi

Here, enabled_any $need_memalign (which is "altivec neon sse") triggers, 
believing altivec and sse to be enabled (while neon correctly seems to be 
disabled).

// Martin