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

Give a better exception when a jar contains same classfile twice. #12093

Merged
merged 1 commit into from Jul 7, 2015

Conversation

rmuir
Copy link
Contributor

@rmuir rmuir commented Jul 7, 2015

And ignore the known issue with xmlbeans for now... though it may
cause us issues ultimately: https://issues.apache.org/jira/browse/XMLBEANS-499

And ignore the known issue with xmlbeans for now... though it may
cause us issues ultimately: https://issues.apache.org/jira/browse/XMLBEANS-499
@rmuir rmuir added the PITA label Jul 7, 2015
@mikemccand
Copy link
Contributor

LGTM ... I had no idea a .zip file was allowed to have exactly the same name more than once!

@rmuir
Copy link
Contributor Author

rmuir commented Jul 7, 2015

I opened https://issues.apache.org/jira/browse/TIKA-1675 just to try to make the issue more well known at least. I am not sure if tika can remove the dependency but the bug has sat unfixed for a long time.

rmuir added a commit that referenced this pull request Jul 7, 2015
Give a better exception when a jar contains same classfile twice.
@rmuir rmuir merged commit 7dbc5c7 into elastic:master Jul 7, 2015
@clintongormley clintongormley added >enhancement :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts v2.0.0-beta1 labels Jul 8, 2015
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >enhancement PITA Team:Delivery Meta label for Delivery team v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants