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

Fixes IOException in jar -x on shadow jars #899

Merged
merged 1 commit into from
Jan 4, 2016
Merged

Conversation

codefromthecrypt
Copy link
Member

The shadowJar plugin makes a single file out of all the dependencies of
our services. Occasionally, this leads to file conflicts.

Jackson's META-INF/LICENSE conflicts with the directory META-INF/license

While we can run the shadowJars, you cannot extract them with jar -x,
as it ends up with an IOException extracting META-INF/license.

This excludes Jackson's META-INF/LICENSE, until we have a better
solution, ideally with one of the proposals in the shadow jar project.

See GradleUp/shadow#86
See GradleUp/shadow#102

The shadowJar plugin makes a single file out of all the dependencies of
our services. Occasionally, this leads to file conflicts.

Jackson's META-INF/LICENSE conflicts with the directory META-INF/license

While we can run the shadowJars, you cannot extract them with `jar -x`,
as it ends up with an IOException extracting META-INF/license.

This excludes Jackson's META-INF/LICENSE, until we have a better
solution, ideally with one of the proposals in the shadow jar project.

See GradleUp/shadow#86
See GradleUp/shadow#102
@codefromthecrypt
Copy link
Member Author

Thanks @joshlong @dsyer for finding the root problem.

@codefromthecrypt
Copy link
Member Author

christmas build is unrelated

codefromthecrypt pushed a commit that referenced this pull request Jan 4, 2016
Fixes IOException in jar -x on shadow jars
@codefromthecrypt codefromthecrypt merged commit 28e8539 into master Jan 4, 2016
@codefromthecrypt codefromthecrypt deleted the unzippable branch January 4, 2016 09:03
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.

1 participant