[MPlayer-dev-eng] [PATCH] fix mencoder compilation with lavf X11 demuxer

[Nicolas Plourde]

On 13-Dec-06, at 6:51 PM, Corey Hickey wrote:

> Hello,
>
> Compilation of mencoder fails for me now; lavf compilation includes  
> some  X11 headers, but mencoder's compilation doesn't link with the  
> corresponding libraries.
>
> The attached patch fixes the problem for me. I don't know if it's  
> the right fix, so I won't threaten to apply it, but it would be  
> nice if somebody took care of this (or told me to apply my patch).
>
> -Corey
> Index: configure
> ===================================================================
> --- configure	(revision 21611)
> +++ configure	(working copy)
> @@ -3843,8 +3843,12 @@
>      else
>        _ld_tmp="$I -lXext -lX11 $_ld_sock $_ld_pthread"
>      fi
> -    cc_check $_inc_x11 $_ld_tmp && _libs_mplayer="$_libs_mplayer  
> $_ld_tmp" \
> -                      && _x11=yes && break
> +    if cc_check $_inc_x11 $_ld_tmp ; then
> +      _libs_mplayer="$_libs_mplayer $_ld_tmp"
> +      _libs_mencoder="$_libs_mencoder $_ld_tmp"
> +      _x11=yes
> +      break
> +    fi
>    done
>  fi
>  if test "$_x11" = yes ; then
>

This is part 1 of the problem

x11grab.c should not be included in the build if x11 is disable.  
Right now compilation fail if you use --disable-x11

====
Nicolas Plourde <nicolas.plourde at gmail.com>