[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