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

Igor Ignatyev iignatyev at openjdk.java.net
Thu Jul 22 20:13:49 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

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

Commit messages:
 - check exit code
 - fixed import
 - reformat code
 - propagate -Djava.library.path

Changes: https://git.openjdk.java.net/jdk17/pull/273/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk17&pr=273&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8271160
  Stats: 9 lines in 1 file changed: 3 ins; 2 del; 4 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