[MPlayer-cvslog] r20527 - trunk/configure

[Guillaume Poirier]

Hi,

diego wrote:
> Author: diego
> Date: Mon Oct 30 10:08:19 2006
> New Revision: 20527
> 
> Modified:
>    trunk/configure
> 
> Log:
> Fix libmpdvdkit2 linking on Mac OS X.
> 
> 
> Modified: trunk/configure
> ==============================================================================
> --- trunk/configure	(original)
> +++ trunk/configure	Mon Oct 30 10:08:19 2006
> @@ -5209,6 +5209,7 @@
>      else
>        if darwin ; then
>         _def_dvd_darwin='#define DARWIN_DVD_IOCTL'
> +       _macosx_frameworks="$_macosx_frameworks -framework IOKit"
>        fi
>      fi
>    fi

Though I trust your judgment regarding how things should be
implemented in configure, shouldn't configure test if IOKit framework
exists, and enable libmpdvdkit2 only it it exists?

Now, maybe it's such a basic lib/framework that we can safely assume
that if you're on OSX, it's always there.... I don't know.


Guillaume