Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

License violations were found #16

Closed
ghost opened this issue Oct 23, 2020 · 3 comments
Closed

License violations were found #16

ghost opened this issue Oct 23, 2020 · 3 comments
Assignees

Comments

@ghost
Copy link

ghost commented Oct 23, 2020

Replicate

Attempted to update the demo application to use JavaFX 11 and Gradle 6.6.1.

  1. Download desktop-pane-fx.zip.
  2. Unzip the archive.
  3. Change to the project directory.
  4. Run: gradle clean build

Expected

Application builds.

Actual

Missing header in: subprojects/desktoppanefx-core/desktoppanefx-core.gradle
> Task :desktoppanefx-core:licenseGradle FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':desktoppanefx-core:licenseGradle'.
> License violations were found: /home/username/dev/java/desktoppanefx/subprojects/desktoppanefx-core/desktoppanefx-core.gradle}

Environment

Liberica JDK 14 (full), Linux (XFCE/X11), IntelliJ 2020 (recent version), Gradle 6.6.1

@aalmiray aalmiray self-assigned this Oct 23, 2020
@aalmiray
Copy link
Collaborator

The file desktoppanefx-core.gradle has the correct license header in the master branch -> https://github.com/kordamp/desktoppanefx/blob/master/subprojects/desktoppanefx-core/desktoppanefx-core.gradle

Did you remove or update the header as part of the upgrade?
This can be fixed by running gradle licenseFormat.

@ghost
Copy link
Author

ghost commented Oct 24, 2020

I wasn't able to get this to build with JavaFX 11, JDK 14, and Gradle 6.6.1. Are there build instructions for such a combination?

@aalmiray
Copy link
Collaborator

I believe the major stumbling block is using the openjfx gradle plugin to resolve the JavaFX dependencies. You can have a look at a project that has the plugin configured at https://github.com/HanSolo/tilesfx/blob/master/build.gradle

Take note that the openjfx plugin will generate a POM with resolved classifiers which is bad for consumers. The TilesFX build has a block that removes said classifiers.

Unfortunately Gradle offers no support for creating multi release JARs without pain which is why the DesktopPaneFX build remains Java 8 compatible for the time being.

@ghost ghost closed this as completed Oct 24, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant