[MPlayer-dev-eng] [PATCH] bugfix vo_png vo_gif89a

[pl]

On Mon, May 13, 2002 at 06:00:11PM -0500, Joey Parrish wrote:
> On Tue, May 14, 2002 at 12:00:37AM +0200, Arpi wrote:
> > anyway, even fi you add -lX11 -lXext etc, there will be another issue -
> > mplayer compiled without X11 support won't compile due to libungif
> Well, libungif won't be linked against X11 unless
> you have X11 support to begin with, right?
> I wish I had a Slackware box to test on, but it
> would seem to me that those who want an X-less
> MPlayer would either 1) not have libungif or
> 2) have a libungif that isn't linked against X11.
> 
> > anyway, afair there is a common $_x11... something containing X11 libs in
> > configure, but i didn't managed to find it, while i remember it's there.
> Arg.  I'll tinker with it some more I guess.
> 
> > maybe pl can help with it
> I welcome all the help I can get.
> Anyone have any suggestions?

The attached patch should test for ungif then ungif+x11 then gif then
gif+x11 before disabling the gif support (not tested & not commited).

-- 
Best regards,
  pl
-------------- next part --------------
Index: configure
===================================================================
RCS file: /cvsroot/mplayer/main/configure,v
retrieving revision 1.470
diff -u -r1.470 configure
--- configure	13 May 2002 20:52:10 -0000	1.470
+++ configure	14 May 2002 07:07:15 -0000
@@ -2345,9 +2345,15 @@
   if cc_check -lungif && "$TMPO" >> "$TMPLOG" ; then
     _gif=yes
     _ld_gif="-lungif"
+  elif cc_check -lungif $_ld_x11 && "$TMPO" >> "$TMPLOG" ; then
+    _gif=yes
+    _ld_gif="-lungif $_ld_x11"
   elif cc_check -lgif && "$TMPO" >> "$TMPLOG" ; then
     _gif=yes
     _ld_gif="-lgif"
+  elif cc_check -lgif $_ld_x11 && "$TMPO" >> "$TMPLOG" ; then
+    _gif=yes
+    _ld_gif="-lgif $_ld_x11"
   fi
 fi