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