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

Maven Shade: exclude MANIFEST and produce shaded SOURCE jar #554

Merged
merged 3 commits into from
Jul 7, 2021

Conversation

brenuart
Copy link
Collaborator

@brenuart brenuart commented Jul 6, 2021

  • exclude MANIFEST from shaded jars to make sure we keep only the one from the project in the final result
  • explicitly configure the plugin to the Disruptor and all the other dependencies (if not, new project dependencies will be shaded as well if the plugin configuration is not updated)
  • produce shaded source as well (handy when debugging applications making use of the library)

Related issue: #553

- exclude MANIFEST from shaded jars to make sure we keep only the one from the project in the final result
- explicitly configure the plugin to <include> the Disruptor and <exclude> all the other dependencies (if not, new project dependencies will be shaded as well if the plugin configuration is not updated)
- produce shaded source as well (handy when debugging applications making use of the library)

Related issue: logfellow#553
@brenuart
Copy link
Collaborator Author

brenuart commented Jul 6, 2021

The plugin still issue some warnings after this change:

[INFO] Minimizing jar net.logstash.logback:logstash-logback-encoder:jar:7.0-SNAPSHOT
[WARNING] /Users/brenuart/Documents/projects/git/logstash-logback-encoder/target/classes (Is a directory)
[WARNING] /Users/brenuart/Documents/projects/git/logstash-logback-encoder/target/classes (Is a directory)

This is caused by the minimizeJar option. It seems to be a known issue (see apache/maven-shade-plugin#83) and should be fixed in future releases (issue title is different but seems to share the same root cause).

…ced pom in the project build directory

Related issue: logfellow#555
…ncy reduced pom in the project build directory"

This reverts commit 26136d5.
@philsttr philsttr merged commit 412932a into logfellow:main Jul 7, 2021
@philsttr philsttr added this to the 7.0 milestone Jul 7, 2021
@brenuart brenuart deleted the gh553 branch July 7, 2021 10:08
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.

None yet

2 participants