Skip to content

Remove JavaFX dependency#215

Open
rnayabed wants to merge 1 commit intoHanSolo:masterfrom
rnayabed:patch-1
Open

Remove JavaFX dependency#215
rnayabed wants to merge 1 commit intoHanSolo:masterfrom
rnayabed:patch-1

Conversation

@rnayabed
Copy link
Copy Markdown

@rnayabed rnayabed commented Sep 21, 2021

Fix #198

While using this plugin, it ignores the user specified JavaFX version and uses the one that this depends on (in this case, 16.0.1 depends on JavaFX 16).

Currently, one needs to do the following in order to use their own specific JavaFX version :

<dependency>
  <groupId>eu.hansolo</groupId>
  <artifactId>medusa</artifactId>
  <version>16.0.1</version>
  <exclusions>
    <exclusion>
      <groupId>org.openjfx</groupId>
      <artifactId>*</artifactId>
    </exclusion>
  </exclusions>
</dependency>

This is the same approach used in ControlsFX

While using this plugin, it ignores the user specified JavaFX version and uses the one that this depends on (in this case, 16.0.1 depends on JavaFX 16).

Currently, one needs to do the following in order to use their own specific JavaFX version :

```
<dependency>
            <groupId>eu.hansolo</groupId>
            <artifactId>medusa</artifactId>
            <version>16.0.1</version>
            <exclusions>
                <exclusion>
                    <groupId>org.openjfx</groupId>
                    <artifactId>javafx-graphics</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.openjfx</groupId>
                    <artifactId>javafx-controls</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.openjfx</groupId>
                    <artifactId>javafx-base</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.openjfx</groupId>
                    <artifactId>javafx-swing</artifactId>
                </exclusion>
            </exclusions>
</dependency>
```
@rnayabed rnayabed changed the title Remove JavaFX dependency Fix #198 Remove JavaFX dependency Sep 21, 2021
@rnayabed rnayabed changed the title Fix #198 Remove JavaFX dependency Remove JavaFX dependency Sep 21, 2021
@rnayabed
Copy link
Copy Markdown
Author

@HanSolo thoughts?

@rnayabed
Copy link
Copy Markdown
Author

rnayabed commented Oct 8, 2021

@HanSolo ???

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove Maven dependencies on javafx

1 participant