On Thu, Jul 19, 2012 at 11:43:32AM +0100, Mans Rullgard wrote:
> This variable is only used once, so setting it there is clearer.
> 
> --- a/configure
> +++ b/configure
> @@ -1975,7 +1975,6 @@ elif $cc -v 2>&1 | grep -qi ^gcc; then
>  elif $cc --version 2>/dev/null | grep -q Intel; then
>      cc_type=icc
>      cc_ident=$($cc --version | head -n1)
> -    icc_version=$($cc -dumpversion)
>      CC_DEPFLAGS='-MMD'
>      AS_DEPFLAGS='-MMD'
> @@ -3149,6 +3148,7 @@ if enabled icc; then
>      # icc 11.0 and 11.1 work with ebp_available, but don't pass the test
>      enable ebp_available
>      if enabled x86_32; then
> +        icc_version=$($cc -dumpversion)
>          test ${icc_version%%.*} -ge 11 &&
>              check_cflags -falign-stack=maintain-16-byte ||

Well, it also makes sense to set this in the compiler section, but
if you prefer it this way, I don't mind...

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to