[MPlayer-cvslog] r35276 - trunk/configure

al subversion at mplayerhq.hu
Sat Oct 27 14:22:27 CEST 2012


Author: al
Date: Sat Oct 27 14:22:26 2012
New Revision: 35276

Log:
build/configure: Move addition of extra CFLAGS further down

The test is know made on cc_vendor after we correctly identified
the compiler. Before we were just assuming if the program is named
cc it is GCC.

For the record:
The condition may still/or in future be wrong for some compilers on
darwin and I doubt the flags are really necessary nowadays.

Patch by Xidorn Quan >quanxunzhen gmail.com<

Modified:
   trunk/configure

Modified: trunk/configure
==============================================================================
--- trunk/configure	Fri Oct 26 21:06:50 2012	(r35275)
+++ trunk/configure	Sat Oct 27 14:22:26 2012	(r35276)
@@ -1577,9 +1577,6 @@ _getch=getch2.c
 
 if darwin; then
   extra_cflags="-mdynamic-no-pic $extra_cflags"
-  if test "$(basename $_cc)" != "clang" ; then
-    extra_cflags="-falign-loops=16 -shared-libgcc $extra_cflags"
-  fi
   _timer=timer-darwin.c
 fi
 
@@ -1725,6 +1722,10 @@ else
 fi # icc
 test "$cc_fail" = yes && die "unsupported compiler version"
 
+if darwin && test "$cc_vendor" != "clang" ; then
+  extra_cflags="-falign-loops=16 -shared-libgcc $extra_cflags"
+fi
+
 echocheck "working compiler"
 cflag_check "" || die "Compiler is not functioning correctly. Check your installation and custom CFLAGS $CFLAGS ."
 echo "yes"


More information about the MPlayer-cvslog mailing list