[MPlayer-users] problem with configure script

[Steven M. Schultz]

On Tue, 17 Jun 2003, D Richard Felker III wrote:

> > if darwin ; then
> >     if [[ `sysctl -n hw.vectorunit` == 1 ]]; then
> >         _altivec=yes
> >     fi
> 
> Yes, whoever added that is clueless about shell scripting. I just
> fixed it in cvs. Sorry about that.

	I see the fix was what I thought it should be :)

	As long as we're on the topic of altivec enabled machines...

	The _altivec variable is clobbered (forced to 'no') during the
	check for <altivec.h>.   This causes G4 based systems which do
	have Altivec instructions (but lack altivec.h - OS/X systems for
	example) to not use them

	Defining another flag "have_altivec_h" would seem to be a solution
	to the problem:

--- configure.dist	Tue Jun 17 09:18:30 2003
+++ configure	Tue Jun 17 09:22:41 2003
@@ -1995,14 +1995,15 @@
 #include <altivec.h>
 int main(void) { return 0; }
 EOF
-_altivec=no
-cc_check && _altivec=yes
-if test "$_altivec" = yes ; then
+
+have_altivec_h=no
+cc_check && have_altivec_h=yes
+if test "$have_altivec_h" = yes ; then
   _def_altivec_h='#define HAVE_ALTIVEC_H 1'
 else
   _def_altivec_h='#undef HAVE_ALTIVEC_H'
 fi
-echores "$_altivec"
+echores "$have_altivec_h"
 else
   _def_altivec_h='#undef HAVE_ALTIVEC_H'
 fi

	What do you think?

	Cheers,
	Steven Schultz