On Mon, Nov 26, 2001 at 12:06:51PM +0100, Wolfram Gloger wrote:
> > First, a check for -lz and using that with -lpng seems prudent.
>
> I disagree, if libpng depends on libz it should have a DT_NEEDED entry
> in it. And for me, it does:
>
> % ldd /usr/lib/libpng.so.2.1.0.5
> libz.so.1 => /usr/lib/libz.so.1 (0x4002b000)
> libm.so.6 => /lib/libm.so.6 (0x4003b000)
> libc.so.6 => /lib/libc.so.6 (0x40058000)
> /lib/ld-linux.so.2 => /lib/ld-linux.so.2 (0x80000000)
They may well be, on fixed up distributions.
However, out of the box, lib png does NOT set that:
nexus@thune[12:50pm]~(501) ldd /usr/lib/libpng.so.3.1.2.0
libm.so.6 => /lib/libm.so.6 (0x4003e000)
libc.so.6 => /lib/libc.so.6 (0x40060000)
/lib/ld-linux.so.2 => /lib/ld-linux.so.2 (0x80000000)
Feel free to send appropriate patches to the libpng maintainers. But until
that happens, DT_NEEDED should not be required.
After all, while xawtv may be linux specific, that doesn't mean that the
configure.in shouldn't be as portable as possible. Because then it becomes
a source of reusable code itself. At least in part.
> > Second, if the user supplies LIBS=, they should be honored in the Makefile,
> > correct?
>
> Hmm, I don't know whether there is a standard for this, it could be
> EXTRA_LIBS or X11_EXTRA_LIBS too..
That IS the standard. At least from my reading of the autoconf manual.
mrc
--
Mike Castle dalgoda@xxxxxxxxxxxxx www.netcom.com/~dalgoda/
We are all of us living in the shadow of Manhattan. -- Watchmen
fatal ("You are in a maze of twisty compiler features, all different"); -- gcc
