RFR: 8252412: [macos11] system dynamic libraries removed from filesystem [v2]

Martin Buchholz martin at openjdk.java.net
Fri Jan 22 20:11:45 UTC 2021


On Fri, 22 Jan 2021 19:46:13 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

>> Martin Buchholz has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR.
>
> src/java.smartcardio/unix/classes/sun/security/smartcardio/PlatformPCSC.java line 132:
> 
>> 130:         // existence of the containing directory instead of the file.
>> 131:         lib = PCSC_FRAMEWORK;
>> 132:         if (new File(lib).getParentFile().isDirectory()) {
> 
> This is outside of my normal area, could you please explain why checking the existence of the containing directory is equivalent to checking the file here? Does it provide the expected behavior on all unix-like platforms or only macos?
> 
> Could you please also provide a jtreg test for this change?

The directory structure is intact - only the file is removed from the filesystem.
More generally, for many frameworks, where there used to be


the file is gone, but 


remains.

I don't think we need a jtreg test, since any functionality associated with PCSC is broken on this platform.  I added label noreg-other

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

PR: https://git.openjdk.java.net/jdk/pull/2119


More information about the core-libs-dev mailing list