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

[Bernd Ernesti]

On Fri, Dec 15, 2006 at 01:14:53AM +0200, Ivan Kalvachev wrote:
[..]

> I've been having problem with linking mencoder to fontconfig (freetype too),
> because when configure test them the cc_check code have the X lib includes
> that happen to be the same as fontconfig/type one (they were part of 
> Xorg6.9-).
> This prevents pgk-config from inserting the correct build and link
> options (again).
> 
> I had discussed this in the dev irc channel. Reimar seems to favour
> using same lib includes
> for configure, mplayer and mencoder. He also dislikes pkg-config (just
> like dalias).

I have the same problem here and thats why I made the attached patch.
But nobody seems to have interest in it. I send a first version on 29 Oct 2006
with the subject "[PATCH] fix extra include and library options in configure".

Bernd

-------------- next part --------------
Index: configure
===================================================================
--- configure	(revision 21621)
+++ configure	(working copy)
@@ -62,9 +62,9 @@
   echo >> "$TMPLOG"
   cat "$1" >> "$TMPLOG"
   echo >> "$TMPLOG"
-  echo "$_cc $CFLAGS $_inc_extra $_ld_static $_ld_extra $_libs_mplayer $_libs_mencoder -o $TMPO $@" >> "$TMPLOG"
+  echo "$_cc $CFLAGS $@ $_inc_extra $_ld_static $_ld_extra $_libs_mplayer $_libs_mencoder -o $TMPO" >> "$TMPLOG"
   rm -f "$TMPO"
-  $_cc $CFLAGS $_inc_extra $_ld_static $_ld_extra $_libs_mplayer $_libs_mencoder -o "$TMPO" "$@" >> "$TMPLOG" 2>&1
+  $_cc $CFLAGS "$@" $_inc_extra $_ld_static $_ld_extra $_libs_mplayer $_libs_mencoder -o "$TMPO" >> "$TMPLOG" 2>&1
   TMP="$?"
   echo >> "$TMPLOG"
   echo "ldd $TMPO" >> "$TMPLOG"
@@ -5378,7 +5378,7 @@
 fi
 if test "$_freetype" = yes ; then
     _def_freetype='#define HAVE_FREETYPE'
-    _inc_extra="$_inc_extra `$_freetypeconfig --cflags`"
+    _inc_extra="`$_freetypeconfig --cflags` $_inc_extra"
     _ld_extra="$_ld_extra `$_freetypeconfig --libs`"
 else
     _def_freetype='#undef HAVE_FREETYPE'