Do not throw exception for missing system modules that are "required static"

Using update4j 1.4.2, I just got this error message when downloading a jar file:

java.lang.IllegalStateException: System module ‘jdk.jfr’ is missing from JVM image, required by ‘javafx.base’

When looking at the module using jmod, we are seeing this:

jmod describe javafx.base.jmod 
javafx.base
exports javafx.beans
exports javafx.beans.binding
exports javafx.beans.property
exports javafx.beans.property.adapter
exports javafx.beans.value
exports javafx.collections
exports javafx.collections.transformation
exports javafx.event
exports javafx.util
exports javafx.util.converter
requires java.base mandated
requires java.desktop
requires jdk.jfr static
qualified exports com.sun.javafx to javafx.controls javafx.fxml javafx.graphics javafx.swing
qualified exports com.sun.javafx.beans to javafx.controls javafx.fxml javafx.graphics
qualified exports com.sun.javafx.binding to javafx.controls javafx.graphics
qualified exports com.sun.javafx.collections to javafx.controls javafx.graphics javafx.media javafx.swing
qualified exports com.sun.javafx.event to javafx.controls javafx.graphics
qualified exports com.sun.javafx.logging to javafx.controls javafx.fxml javafx.graphics javafx.swing javafx.web
qualified exports com.sun.javafx.property to javafx.controls
qualified exports com.sun.javafx.reflect to javafx.fxml javafx.web
qualified exports com.sun.javafx.runtime to javafx.graphics
contains com.sun.javafx.logging.jfr
contains com.sun.javafx.property.adapter

The important line here is requires jdk.jfr static. As we can see, it is a “static” requirement. According to this article, “requires static” is used for compile-only dependencies:

• At compile time, bar must be present or there will be an error. During compilation bar is readable by foo.
• At run time, bar might be absent and that will cause neither error nor warning. If it is present, it is readable by foo.

So, I think update4j should not throw an exception in cases like these.

[ChristianCiach]

Well, yeah, all my coworkers already know about update4j because I did an in-depth presentation about how the updater works. Other than that, I don’t have contacts to any other Java developers :p

[ChristianCiach]

Thanks again. I can confirm that the fix works.