[jdk17] RFR: 8271160: runtime/jni/checked/TestCheckedJniExceptionCheck.java doesn't set -Djava.library.path [v2]
Igor Ignatyev
iignatyev at openjdk.java.net
Thu Jul 22 22:19:34 UTC 2021
> 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
Igor Ignatyev has updated the pull request incrementally with one additional commit since the last revision:
check that TEST_START has been met
-------------
Changes:
- all: https://git.openjdk.java.net/jdk17/pull/273/files
- new: https://git.openjdk.java.net/jdk17/pull/273/files/ff884103..c95674f6
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk17&pr=273&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk17&pr=273&range=00-01
Stats: 3 lines in 1 file changed: 3 ins; 0 del; 0 mod
Patch: https://git.openjdk.java.net/jdk17/pull/273.diff
Fetch: git fetch https://git.openjdk.java.net/jdk17 pull/273/head:pull/273
PR: https://git.openjdk.java.net/jdk17/pull/273
More information about the hotspot-runtime-dev
mailing list