<AWT Dev> RFR: Allow using a system installed libpng

[Magnus Ihse Bursie]

On 2014-02-10 18:43, Andrew Hughes wrote:
> You're already using it:
>
> PKG_CHECK_MODULES([LIBFFI], [libffi])
>
> Why that's in LIB_SETUP_STATIC_LINK_LIBSTDCPP, I have no idea.
Because libraries.m4 is in need of a long overdue cleanup. :-(
>
>> * Second, I believe the original intention was to allow for a third
>> option, --with-libX=<dir>, which would point to an location in which the
>> library is installed, similar to how e.g. --with-alsa works.
>>
> Yes, but you don't allow that:
>
>      AC_MSG_ERROR([Invalid value for --with-zlib: ${with_zlib}, use 'system' or 'bundled'])
>
> so using --with-x is confusing if someone does specify a directory.
Yes. As I said, that was the original intention -- not the current 
implementation. Once again, the code in libraries.m4 is in dire need of 
some TLC. Getting it in better shape *is* on my agenda, but it tends to 
be pushed down all the time.

So from my point of view, Omair's patch is good. It provides additional 
value. It does not solve all problems in libraries.m4, nor is it the 
complete answer on how --with-libpng will behave in the future. But it 
is a good step on the way. I'm willing to sponsor the patch. But I won't 
do that if you object to accepting it. (After all, from Oracle's point 
of view there's no real need for this patch.) I'm also not very much 
interested in working out this specific patch to perfection, when the 
whole of libraries.m4 needs so much work. So, it's a bit of "take it or 
leave it".

To be extremely clear: Andrew, do you object to bringing Omairs patch, 
as it is, into OpenJDK?

/Magnus