RFR: 7134690: remove legacy jnilib support from ClassLoader and System [macosx]

[Scott Kovatch]

On Jan 27, 2012, at 4:12 AM, Alan Bateman wrote:

> On 27/01/2012 11:41, Michael McMahon wrote:
>> Can I get the following change reviewed please? The change is to remove
>> some mac specific code from:
>> 
>> src/share/classes/java/lang/System.java and
>> src/share/classes/java/lang/ClassLoader.java
>> 
>> which added support for non-standard native library suffixes on Mac OS
>> (ie. .jnilib as well as the standard .dylib).
>> 
>> We would like to remove this code, so there will be no changes to those sources
>> when the Mac changes get pushed to jdk7u-dev
>> 
>> I have submitted a hotspot CR to track providing the same functionality
>> in Mac specific code in hotspot. Though we can probably live without it in the short-term.
>> 
>> http://cr.openjdk.java.net/~michaelm/7134690/webrev.1/
> This looks fine to me as this wasn't the right place to support this. I don't know how common .jnilib was to know if the VM changes will be required in the short term.

For 7u4 this is probably going to be okay, but we definitely need this functionality for 7u6 when the plugin is released. There are some popular web sites (Minecraft, for example) that use JOGL, which uses a .jnilib extension.

-- Scott K.

----------------------------------------
Scott Kovatch
scott.kovatch at oracle.com
Santa Clara/Pleasanton, CA