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

changed implementation of additionalBundlerResources #239

Merged

Conversation

FibreFoX
Copy link
Member

@FibreFoX FibreFoX commented Oct 5, 2016

Now it is possible to create folders having the bundlerID, and when possible copying directly into these, without loosing the older behavior.
Setting:
<additionalBundlerResources>src/main/deploy/bundlers</additionalBundlerResources>
and having:
src/main/deploy/bundlers/[bundlerid]/additionalFile1 ....

Now it is possible to create folders having the bundlerID, and when possible copying directly into these, without loosing the older behavior.
@FibreFoX
Copy link
Member Author

FibreFoX commented Oct 5, 2016

This is intended to fix #235

@FibreFoX
Copy link
Member Author

FibreFoX commented Oct 5, 2016

available as 8.7.1-SNAPSHOT

@FibreFoX
Copy link
Member Author

FibreFoX commented Oct 5, 2016

DO NOT MERGE ! needs rewrite for the mac-parts ...

The first idea was too naive, so this needed to be reimplemented completely. This now gets worked around and refactored a lot. Additionally the gradle-plugin got some more options, especially the options `failOnError`, `onlyCustomBundlers` and `skipJNLP` got added. The later one mostly makes sense in combination with `failOnError`, because the JNLP-bundler behaves different. Another important change is the switch default of the `checkForAbsolutePaths` option, which will get removed on the next release (I do know at least one person, who will be happy about this decision).

This got migrated from FibreFoX/javafx-gradle-plugin@65cef6d
@FibreFoX
Copy link
Member Author

FibreFoX commented Oct 7, 2016

fixes #238 too

@FibreFoX
Copy link
Member Author

Needs some IT-projects for having implementations regarding #238 being fully tested

@FibreFoX
Copy link
Member Author

This will be worked on soon ;) I'm in the preparation for a lot of mac-specific review until the end of this year.

* moved detection-stuff to Workarounds-class
* fixed our own "mac.app" detecting additional bundler files (now using "mac.app"-subfolder)
bumped version because there are some more additions upcoming
@FibreFoX
Copy link
Member Author

FibreFoX commented Jan 7, 2017

available as 8.8.0-SNAPSHOT

@FibreFoX FibreFoX force-pushed the rewrite_additionalBundlerResources branch from 5361b94 to dad5084 Compare January 7, 2017 22:34
@FibreFoX FibreFoX merged commit 1abf905 into javafx-maven-plugin:master Jan 18, 2017
@FibreFoX FibreFoX deleted the rewrite_additionalBundlerResources branch January 3, 2018 22:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant