On Fri, 28 Jul 2023 09:11:07 GMT, Jose Pereda <jper...@openjdk.org> wrote:

> This PR adds a file lock to the cache directory, allowing access from 
> multiple processes (that is, from more than one JVM) to that directory. 
> 
> This helps solving the issue where the cache is empty or doesn't exist yet, 
> and two JavaFX applications start at the same time, and both try to copy the 
> same native libraries to the same cache.
> 
> No tests have been added to the PR though, since this requires a complex 
> scenario: building the SDK and publishing to local Maven repository, creating 
> two simple JavaFX applications that use those artifacts, using a temporary 
> folder for cache (via `javafx.cachedir`, and launching in parallel both 
> applications.
> 
> I've tested successfully such scenario on Linux, Mac and Windows.

This change fixes the problem at hand, though I'd suggest one more follow up 
ticket (to close the input stream after Files.copy()).

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

Marked as reviewed by angorya (Reviewer).

PR Review: https://git.openjdk.org/jfx/pull/1188#pullrequestreview-1579057424

Reply via email to