[MPlayer-cvslog] r26095 - trunk/configure

Rich Felker dalias at aerifal.cx
Mon Feb 25 20:19:34 CET 2008


On Sun, Feb 24, 2008 at 11:41:39PM +0100, diego wrote:
> Author: diego
> Date: Sun Feb 24 23:41:39 2008
> New Revision: 26095
> 
> Log:
> Disable internal VIDIX on OS/2, patch by Dave Yeo, dave.r.yeo gmail com.
> 
> 
> Modified:
>    trunk/configure
> 
> Modified: trunk/configure
> ==============================================================================
> --- trunk/configure	(original)
> +++ trunk/configure	Sun Feb 24 23:41:39 2008
> @@ -4257,7 +4257,7 @@ if test "$_vidix_internal" = auto ; then
>    x86 && _vidix_internal=yes
>    ppc && linux && _vidix_internal=yes
>    alpha && linux && _vidix_internal=yes
> -  qnx || beos || darwin && _vidix_internal=no
> +  qnx || beos || darwin || os2 && _vidix_internal=no

IMO this approach is backwards. Vidix should be disabled by default
and only enabled on exact cpu/os/toolchain combinations where it is
KNOWN to work. x86 && _vidix_internal=yes is extremely broken..

Rich



More information about the MPlayer-cvslog mailing list