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

List:       openjdk-openjfx-dev
Subject:    Integrated: 8273089: Deprecate JavaFX GTK 2 library for removal
From:       Kevin Rushforth <kcr () openjdk ! java ! net>
Date:       2021-12-15 22:35:04
Message-ID: CBnOep3RCBQiySDVY6Uc4jVMJEqeJlVvz32TSC32KeY=.b22a6096-fae3-4b48-8eaa-cfacbd1b843c () github ! com
[Download RAW message or body]

On Tue, 30 Nov 2021 13:26:30 GMT, Kevin Rushforth <kcr@openjdk.org> wrote:

> This PR deprecates the JavaFX glass GTK 2 library for removal. Since there isn't an \
> associated API, the mechanism for doing this is to print a "deprecated for removal" \
> warning message when the `glassgtk2` library it is loaded. A message is printed \
> regardless of why the GTK 2 library was loaded, which can happen for one of the \
> following reasons: 
> 1. The application or end user has explicitly requested GTK 2 using the \
> `jdk.gtk.version` system property. 2. Another native library has already loaded the \
> GTK 2 library (an application cannot load both the GTK 2 and GTK 3 libraries in the \
> same process) 3. As a fallback if the gtk3 libraries are not available on the \
> system. 
> The warning messages are patterned after the security manager deprecation messages \
> added as part of [JEP 411](https://openjdk.java.net/jeps/411). 
> I added two system tests, one that sets the `jdk.gtk.version` system property to \
> `2` and verifies that the warning message is printed, and one that doesn't set \
> `jdk.gtk.version` and verifies that no warning message is printed.

This pull request has now been integrated.

Changeset: 27f28102
Author:    Kevin Rushforth <kcr@openjdk.org>
URL:       https://git.openjdk.java.net/jfx/commit/27f281028fde507e5f0fe81247133816db7dc41d
                
Stats:     221 lines in 4 files changed: 221 ins; 0 del; 0 mod

8273089: Deprecate JavaFX GTK 2 library for removal

Reviewed-by: pbansal, jvos

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

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


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

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