RFR: JDK-8319516 AIX System::loadLibrary needs support to load a shared library from an archive object [v17]

Suchismith Roy sroy at openjdk.org
Tue Apr 9 10:05:15 UTC 2024


On Mon, 8 Apr 2024 19:45:14 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> Suchismith Roy has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   test change
>
> test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java line 36:
> 
>> 34:         String libraryName = "awt";
>> 35:         File awtSharedObjectPath = new File("/test/lib/libawt.so");
>> 36:         File awtArchivePath = new File("/test/lib/libawt.a");
> 
> How does this work? Did you create a "/test" directory? I don't have it on my machine.

I am not 100% sure. I think it is a temporary directory under jtreg. I referred to the other test under loadLibrary. @mlchung  Could you clarify this ?

> test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java line 37:
> 
>> 35:         File awtSharedObjectPath = new File("/test/lib/libawt.so");
>> 36:         File awtArchivePath = new File("/test/lib/libawt.a");
>> 37:         awtSharedObjectPath.renameTo(awtArchivePath);
> 
> This should work for this test. But, what if an AWT test gets executed after your test? I think copy is safer than renaming.

Maybe i need to rename it back. If we just copy, it would take the .so itself, so we cant enforce it.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1557366150
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1557365063


More information about the core-libs-dev mailing list