-
Notifications
You must be signed in to change notification settings - Fork 76
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
Update Apache Tika #6826
Update Apache Tika #6826
Conversation
core/pom.xml
Outdated
@@ -646,7 +646,7 @@ | |||
<dependency> | |||
<groupId>org.apache.tika</groupId> | |||
<artifactId>tika-core</artifactId> | |||
<version>1.28.5</version> | |||
<version>2.9.2</version> | |||
<exclusions> | |||
<exclusion><!-- excluded because we want to ignore this (1.2.2) version! this dependency handles javax.activation.* imports, and is NOT compatible with the IBM JRE --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure this exclusion is still needed? In the above aspose/pom.xml
, you've removed it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment is a bit out of date, but the version/scope etc are locked in the parent pom.xml 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but I was just thinking that this library isn't a dependency of the tika-core anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I'll remove the comment and exclusion
Quality Gate passedIssues Measures |
upgrade Tika dependency