# HG changeset patch # User diego # Date 1184670975 0 # Node ID 27fc683b631cac9dc368a3a0b26afc71b42edff9 # Parent 99a68d91cfef037d047dc64c96de4a323f14fcdd Modify X11 headers check so as not to add /usr/include, the default system include path, to our custom include path or duplicate entries in it. diff -r 99a68d91cfef -r 27fc683b631c configure --- a/configure Tue Jul 17 11:08:41 2007 +0000 +++ b/configure Tue Jul 17 11:16:15 2007 +0000 @@ -3748,24 +3748,28 @@ echocheck "X11 headers presence" - for I in `echo $_inc_extra | sed s/-I//g` /usr/X11/include /usr/X11R6/include /usr/include/X11R6 /usr/include /usr/openwin/include ; do + _x11_headers="no" + _res_comment="check if the dev(el) packages are installed" + for I in `echo $_inc_extra | sed s/-I//g` /usr/include ; do if test -f "$I/X11/Xlib.h" ; then - _inc_x11="-I$I" + _x11_headers="yes" + _res_comment="" + break + fi + done + for I in /usr/X11/include /usr/X11R6/include /usr/include/X11R6 /usr/openwin/include ; do + if test -f "$I/X11/Xlib.h" ; then + _inc_extra="$_inc_extra -I$I" _x11_headers="yes" _res_comment="using $I" break fi done - if test -z "$_inc_x11" ; then - _x11=no - _x11_headers="no" - _res_comment="check if the dev(el) packages are installed" - fi echores "$_x11_headers" echocheck "X11" -if test "$_x11" != no ; then +if test "$_x11" = auto && test "$_x11_headers" = yes ; then cat > $TMPC < #include @@ -3778,13 +3782,11 @@ else _ld_tmp="$I -lXext -lX11 $_ld_pthread" fi - cc_check $_inc_x11 $_ld_tmp && _libs_mplayer="$_libs_mplayer $_ld_tmp" \ + cc_check $_ld_tmp && _libs_mplayer="$_libs_mplayer $_ld_tmp" \ && _x11=yes && break done fi if test "$_x11" = yes ; then - #FIXME: This is ugly as it can duplicate a -I parameter.. - _inc_extra="$_inc_extra $_inc_x11" _def_x11='#define HAVE_X11 1' _vosrc="$_vosrc x11_common.c vo_x11.c vo_xover.c" _vomodules="x11 xover $_vomodules"