Allow using a system-installed giflib
Omair Majid
omajid at redhat.com
Thu Mar 14 23:39:37 UTC 2013
Hi,
Updated webrev at:
http://cr.openjdk.java.net/~omajid/webrevs/system-giflib/01/
The webrev borrows some idioms that Andrew Hughes pointed out, and uses
system headers for giflib too.
I checked this by building --with-libgif=system and also by removing the
src/share/native/sun/awt/giflib dir and using --with-giflib=system.
On 03/08/2013 12:26 PM, Phil Race wrote:
> If I understand correctly, this removes the directory containing
> the JDK's copy of giflib sources from the set of locations to be
> compiled etc, and replaces it with just a link line pointer to use
> "libgif" which is then expected to be on the default linker path,
> ie in /usr/lib.
>
> I think this is fine except I would guard USE_EXTERNAL_LIBGIF
> with an expectation that this is at least "not" windows, since
> I'm pretty sure that option would always be a mistake there.
The updated webrev does that. I don't have a windows machine to test
this on, unfortunately.
Any comments or suggestions?
Thanks,
Omair
--
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681
More information about the build-dev
mailing list