RFR: JDK-8248667 Need support for building native libraries located in the test/lib directory
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Thu Jul 2 15:48:12 UTC 2020
Sorry if you get this twice, but I seem to have problem reaching the mailing list.
> On 2020-07-02 16:26, Chris Plummer wrote:
> Hi Mangus,
>
> Can you leave out libLingeredApp.c. It will conflict with the version I plan on pushing.
Chris,
You can just remove it, or replace the contents with your file. The code would otherwise have needed additional guards to handle the special case of no files, which was unnecessary to add for a short transition.
/Magnus
>
> thanks,
>
> Chris
>> Chris has requested help to add native test code to the test lib. From
>> the bug report:
>>
>> "As part of the work for JDK-8248194 I'm adding a native method to
>> LingeredApp.c. The native method will be located in:
>>
>> test/lib/jdk/test/lib/apps/libLingeredApp.c
>>
>> However, currently there is only support for building native libs in
>> test/jdk and test/hotspot/jtreg. Support is now also needed for test/lib"
>>
>> This patch adds a simple lib. Since jtreg does not accept multiple roots
>> for native libraries, I have to copy this to both hotspot and jdk native
>> libs in the test image. The change in BuildTestLib.gmk is strictly not
>> necessary, but allows more (unfortunately not all) of the testlib to be
>> compiled, including LingeredApp, which allowed me to generate a proper
>> .h file to base libLingeredApp.c on.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8248667
>> WebRev:
>> http://cr.openjdk.java.net/~ihse/JDK-8248667-build-testlib-native/webrev.01
>>
>> /Magnus
More information about the build-dev
mailing list