[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Integrated: 8222455: JavaFX error loading glass.dll from cache
From:       Marius Hanl <mhanl () openjdk ! java ! net>
Date:       2021-10-30 20:40:27
Message-ID: 0vhC-Ri43KACIjZuALAzNUapdQLamPR_uOlDMyXr8WQ=.14e9f1bf-0213-4b21-bbb1-cc062a8f0916 () github ! com
[Download RAW message or body]

On Tue, 26 Oct 2021 12:09:19 GMT, Marius Hanl <mhanl@openjdk.org> wrote:

> This problem can happen when using multiple instances with different jfx early \
> access (ea) versions. 
> Example: 
> Instance 1 uses 18-ea+4 and Instance 2 uses 18-ea+1. 
> Instance 1 is started (and will cache and use libraries), then instance 2. 
> Now instance 2 detects via a hash comparison that the cached libraries are not the \
> same as the supplied ones.  With this information instance 2 tries to delete the \
> old libraries but fails while doing so (as instance 1 uses them currently) and will \
> terminate right after. 
> The problem here is that instance 1 and 2 are using the same cache folder: \
> **18-ea**. This is because the `NativeLibLoader` uses the `javafx.version` property \
> for determining the folder name, which in case of an ea version will always be \
> **18-ea** (for all ea versions starting with 18 obviously). 
> Fix as also mentioned in the ticket is to use the `javafx.runtime.version` property \
> instead.  With this the folders will be named 18-ea+1 and 18-ea+4.

This pull request has now been integrated.

Changeset: 6c881063
Author:    Marius Hanl <mhanl@openjdk.org>
Committer: Kevin Rushforth <kcr@openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/6c8810634ec63af8116dd978656805b985eec800
                
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8222455: JavaFX error loading glass.dll from cache

Reviewed-by: jvos, kcr

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

PR: https://git.openjdk.java.net/jfx/pull/654


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic