[MPlayer-dev-eng] [PATCH] modify X11 lib path search for mixed 32/64 bit case

Ivan Kalvachev ikalvachev at gmail.com
Fri Jun 3 00:32:19 CEST 2005


On 6/1/05, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:
> Hi,
> next try...
> 
> On Wed, Jun 01, 2005 at 02:50:25PM +0300, Ivan Kalvachev wrote:
> > 1. If user gives x11libdir and the autodetection fails then the we
> > should keep the given by user dir. (it have the power of force, or it
> > would have the power of force if x11=yes)
> 
> Huh? When a a list of possible X11 directories is specified,
> autodetection is done on these, and if it fails it will use the last
> one.
> 
> > I guess that adding _x11_paths and keeping  _ld_x11 would do the trick, but
> > actually handling _x11=no case could be better:
> > +      break
> > +    fi
> > +  done
> > +else
> > +  I=$_x11_paths
> 
> Why setting I here? It is not used at all with --disable-x11, and even
> in the other cases only for the status message...

Oopsss, my bad.
The problem I am thinking of, is when --enable-x11 is used to force
x11 and skip autodetection.
as we have _x11=yes, we will try to find the X11 dir. So we will have
autodetection of directory.  If it cannot find directory it will
disable x11 build, so in effect we overide the force.

So I guess that if developer forces detection then we should use the
library path without checking it (it could be the reason detection to
fail)

Let me try again

@@somewhere
 echocheck "X11"
 if test "$_x11" != no ; then
+  if test "$_x11" = yes && test -n "$_x11_paths" ; then
+       $I=_x11_paths 
+  else
@@do_detection here...
+  fi
 fi
 if test "$_x11" = yes ; then
   _def_x11='#define HAVE_X11 1'
@@end

The rest is OK.

   Ivan Kalvachev
  iive




More information about the MPlayer-dev-eng mailing list