Remove the client's JavaFX dependency after #437
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- Medio día
- Aptitud para principiantes
- 78/100
Línea de trabajo
Wait until #437 is merged, then drop JavaFX leftovers the issue lists: Platform.setImplicitExit(false) in Frame.java, the JavaFX check in ClientPrerequisites, javafx.controls,javafx.fxml in --add-modules in client/build.gradle, the committed client/lib/openjfx.jar, and the OpenJFX credit in AboutMirth.java plus the OpenJFX section of THIRD-PARTY-README.txt. Done when the client builds and starts on a Java 17+ JRE with no JavaFX and those files no longer mention OpenJFX.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
#437 replaces the WebView-based tag field with Swing. That tag field was the only JavaFX user in the Administrator client: #277 (crash on JREs without JavaFX) came from its JFXPanel, and #278 added the startup check to give a clearer error. Once #437 merges, these JavaFX leftovers can go:
Platform.setImplicitExit(false)inFrame.java: only kept the JavaFX runtime alive between tag fields.- The JavaFX check in
ClientPrerequisites(#278). javafx.controls,javafx.fxmlin--add-modulesinclient/build.gradle.- The committed
client/lib/openjfx.jar(OpenJFX 11 classes, GPL-2.0 with Classpath Exception). - The OpenJFX credit in
AboutMirth.javaand the OpenJFX section ofTHIRD-PARTY-README.txt.
Benefits:
- Removes a committed binary and an item the Eclipse Dash License Tool flags as restricted (OpenJFX has no license data in ClearlyDefined).
- The client runs on any Java 17+ runtime, and JavaFX not required.
- Lenguaje dominante
- Java
- Estrellas
- 227
- Forks
- 81
- Merge medio
- 3 d 15 h
- PR fusionados (30 d)
- 13
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OpenIntegrationEngine/engine
-
Unrecognized database schema version is treated as a pre-2.0 databasePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
OpenIntegrationEngine/engine#496 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
OpenIntegrationEngine/engine#485 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
OpenIntegrationEngine/engine#478 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
OpenIntegrationEngine/engine#355 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
OpenIntegrationEngine/engine#347 ·
Los mantenedores suelen responder en 1 día
Todos los issues de OpenIntegrationEngine/engine
Issues similares
-
False Negative: InnerTypeLastCheck does not report members declared after inner enums or annotationsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
checkstyle/checkstyle#22027 ·
Los mantenedores suelen responder en 1 día
-
bug follow-up lane:auto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Sanne/incus-spawn#1497 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
[CI] NettyClientTest.testServerDisconnection may fail with DisconnectExceptionPosiblemente ocupada @yangshangqing95 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100