RFR: Allow using a system installed libpng

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Mon Feb 3 19:41:52 UTC 2014


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 
libraries.m4:
       with_libpng=${DEFAULT_libpng}

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.

/Magnus



More information about the build-dev mailing list