<AWT Dev> RFR: Allow using a system installed libpng
gnu.andrew at redhat.com
Wed Feb 5 00:26:00 UTC 2014
----- Original Message -----
> On 2014-02-03 18:43, Omair Majid wrote:
> > Hi,
> > The following webrevs modify the build system to allow building against
> > the system-installed copy of libpng as well as using the bundled copy of
> > libpng
> > ROOT: http://cr.openjdk.java.net/~omajid/webrevs/system-libpng/01/
> > JDK: http://cr.openjdk.java.net/~omajid/webrevs/system-libpng/01-jdk/
> > A new configure option `--with-libpng` is introduced which defaults to
> > `bundled` but can be set to `system` to use the system-installed copy of
> > libpng.
> Thank you for the patch!
> Looks good to me (but I'm no formal reviewer), apart from this line in
> which should be DEFAULT_LIBPNG, I assume, otherwise the default case
> will not work.
> Due to this, I'd like to ask you to please double check that
> --with-libpng=system | bundled | invalid-value | <empty>, and no flag at
> all, all work as expected.
In this respect, why does this not just use AC_ARG_ENABLE as there are only
two options here (system libpng or bundled libpng)?
Also, this patch hardcodes the libpng cflags and ldflags when these should
be obtained from pkg-config. This is how these values are obtained in IcedTea
and using this patch would thus be a regression.
Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F 8F91 3B96 A578 248B DC07
More information about the build-dev