[Rev 03] RFR: 8238755: allow to create static lib for javafx.media on linux

Kevin Rushforth kcr at openjdk.java.net
Thu Feb 20 22:06:06 UTC 2020


On Thu, 20 Feb 2020 12:19:07 GMT, Johan Vos <jvos at openjdk.org> wrote:

>> * add support for the "resource" protocol (which is used in the GraalVM URLs pointing to statically bundled resources)
>> * avoid duplicate symbols with different gst plugins
>> * statically register gst plugins
> 
> The pull request has been updated with 4 additional commits.

Looks good to me. I ran a full test on Linux with the default dynamic libraries and all looks good. I also tested that the `resource` protocol throws an informative exception:
MediaException: MEDIA_INACCESSIBLE : unknown protocol: resource

I asked one question, but whatever you decide is fine with me.

modules/javafx.base/src/main/java/com/sun/javafx/PlatformUtil.java line 74:

> 73:     private static final boolean IOS = os.startsWith("iOS");
> 74:     private static final boolean STATIC_BUILD = "Substrate VM".equals(System.getProperty("java.vm.name"));
> 75: 

Is it always going to be the case that STATIC_BUILD == is SubStrate VM?

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

Marked as reviewed by kcr (Lead).

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


More information about the openjfx-dev mailing list