On 1/27/12 7:56 AM, Scott Kovatch wrote:
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.
IMHO, this is enough to justify support for .jnilib in 7u4.
Otherwise beta plugin bundle will not work for most popular sites ...

-igor

-- Scott K.

----------------------------------------
Scott Kovatch
scott.kova...@oracle.com
Santa Clara/Pleasanton, CA



Reply via email to