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 pull request has now been integrated.

Changeset: 72f05a79
Author:    Jose Pereda <jper...@openjdk.org>
URL:       
https://git.openjdk.org/jfx/commit/72f05a79bf66e39848e491320447997e54625617
Stats:     20 lines in 1 file changed: 18 ins; 1 del; 1 mod

8307536: FileAlreadyExistsException from NativeLibLoader when running 
concurrent applications with empty cache

Reviewed-by: kcr, angorya, jvos

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

PR: https://git.openjdk.org/jfx/pull/1188

Reply via email to