[jdk17] Integrated: 8271160: runtime/jni/checked/TestCheckedJniExceptionCheck.java doesn't set -Djava.library.path

[Igor Ignatyev]

On Thu, 22 Jul 2021 20:07:09 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> Hi all,
> 
> could you please review this small patch that sets `-Djava.library.path` to ` test.nativepath` for `TestCheckedJniExceptionCheck`'s child JVM so it will be able to locate the library?
> 
> from JBS:
>> TestCheckedJniExceptionCheck spawns a child JVM that uses a test native library, TestCheckedJniExceptionCheck, but it doesn't add/set test.nativepath value to -Djava.library.path, so the child JVM can't find the library and fails. 
>>
>> this issue hasn't been noticed early b/c TestCheckedJniExceptionCheck doesn't check the exit code of the child JVM.
> 
> Thanks,
> -- Igor

This pull request has now been integrated.

Changeset: 2d165a2b
Author:    Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jdk17/commit/2d165a2bdf6efbfb1541af53d83492769bc0f616
Stats:     12 lines in 1 file changed: 6 ins; 2 del; 4 mod

8271160: runtime/jni/checked/TestCheckedJniExceptionCheck.java doesn't set -Djava.library.path

Reviewed-by: dcubed, dholmes

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

PR: https://git.openjdk.java.net/jdk17/pull/273