On Tue, 26 Mar 2024 17:48:49 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:
>> When resolving the location of the NSS library, the code appended a >> hard-coded path: `/nss/lib`. This makes it difficult for a person to specify >> the library when it is installed in a location such as `/usr/lib64`. >> >> In this PR, I removed the hard-coded path and updated the code to search the >> specified path for the nss library file, softokn3. > > Matthew Donovan has updated the pull request incrementally with one > additional commit since the last revision: > > added try-with-resources to Stream and cleaned up path logic Looks good to me. I verified that it fixes the problem when running PKCS11 tests with system libraries (e.g. /usr/lib64). ------------- Marked as reviewed by mbalao (Committer). PR Review: https://git.openjdk.org/jdk/pull/18488#pullrequestreview-1961396374
