[7u4] RFR: 7134701 [macosx] Support legacy native library names

[Michael McMahon]

Sorry Scott, that was the wrong link. I mistakenly forwarded the original
email instead of the updated one..

This is the correct webrev:

http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.2/

Thanks,
Michael.


On 02/04/12 17:20, Scott Kovatch wrote:
>
> On Apr 2, 2012, at 9:35 AM, Michael McMahon wrote:
>
>> This should have been reviewed on this list. So, I am forwarding the
>> request again. It hasn't changed since the original webrev was reviewed
>> on core-libs and the macos lists.
>>
>> It is the same code change as for jdk 8
>>
>> http://cr.openjdk.java.net/~michaelm/7134701/webrev.7u4.1/ 
>> <http://cr.openjdk.java.net/%7Emichaelm/7134701/webrev.7u4.1/>
>
> So in the end it looks like we will only support checking both .jnilib 
> and .dylib on Mac:
>
> 1. When using System.loadLibrary()
> 2. The class calling System.loadLibrary() was NOT loaded from a custom 
> class loader
> 3. isAbsolute is false, which implies System.load() was called.
>
> I saw the macosx discussion, and I think this is fine. If you're 
> calling System.load() you know what file you want to load, and the 
> extension shouldn't be inferred. Likewise, we shouldn't override the 
> policy of a non-system class loader.
>
> I enthusiastically approve this change. :-)
>
> -- Scott K.
>
> ----------------------------------------
> Scott Kovatch
> scott.kovatch at oracle.com <mailto:scott.kovatch at oracle.com>
> Santa Clara/Pleasanton, CA
>