Re: RFR: Allow using a system-installed lcms2

2014-02-28 Thread Omair Majid
* Omair Majid [2014-02-20 17:43]: > The following is a preliminary webrev that allows OpenJDK to build and > run against a system-installed copy of lcms2 rather than the copy > bundled with OpenJDK: > > root: http://cr.openjdk.java.net/~omajid/webrevs/system-lcms/00/ > jdk: http://cr.openjdk.java

Re: RFR: Allow using a system-installed lcms2

2014-02-25 Thread Omair Majid
* Omair Majid [2014-02-21 10:50]: > * Andrew Hughes [2014-02-21 10:36]: > > > I think the "j2" convention is reasonable. > > > > This is where we disagree. I may have agreed if this was new, but we've been > > using libjavalcms.so for the lifetime of 7 and I see no reason to change > > this. >

Re: RFR: Allow using a system-installed lcms2

2014-02-21 Thread Andrew Hughes
- Original Message - > On 2014-02-20 23:40, Omair Majid wrote: > > Hi, > > > > The following is a preliminary webrev that allows OpenJDK to build and > > run against a system-installed copy of lcms2 rather than the copy > > bundled with OpenJDK: > > > > root: http://cr.openjdk.java.net/~oma

Re: RFR: Allow using a system-installed lcms2

2014-02-21 Thread Omair Majid
* Andrew Hughes [2014-02-21 10:36]: > > I think the "j2" convention is reasonable. > > This is where we disagree. I may have agreed if this was new, but we've been > using libjavalcms.so for the lifetime of 7 and I see no reason to change this. > > Also, "j2" seems pretty meaningless. I had the

Re: RFR: Allow using a system-installed lcms2

2014-02-21 Thread Omair Majid
* Magnus Ihse Bursie [2014-02-21 08:21]: > On 2014-02-20 23:40, Omair Majid wrote: > >1. The sources for the bundled library are contained next to > >OpenJDK-specific sources. This is not true for the bundled copies of > >zlib, libpng and giflib. On the other hand, the jpeg code in OpenJDK > >also

Re: RFR: Allow using a system-installed lcms2

2014-02-21 Thread Magnus Ihse Bursie
On 2014-02-20 23:40, Omair Majid wrote: Hi, The following is a preliminary webrev that allows OpenJDK to build and run against a system-installed copy of lcms2 rather than the copy bundled with OpenJDK: root: http://cr.openjdk.java.net/~omajid/webrevs/system-lcms/00/ jdk: http://cr.openjdk.java