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

Circular dependencies with tika-core #24

Open
michael-simons opened this issue Feb 1, 2018 · 3 comments
Open

Circular dependencies with tika-core #24

michael-simons opened this issue Feb 1, 2018 · 3 comments

Comments

@michael-simons
Copy link

Hi there,
I'm using maven-enforcer-plugin to make sure I don't have mismatching dependencies in my build.

vorbis-java-tika depends on an outdated version of tika-core and pulling it in through tika-parsers leads to having a conflict between tika 1.17 and 1.12 currently.

ohne titel

@Gagravarr
Copy link
Owner

The Tika Core dependency is set to the oldest one that the code is able to work with, that way people stuck on older versions of Tika are still able to use the latest release.

If there's some Maven magic which better expresses "this is the oldest version you can work with and should be used at compile time, but newer versions should be used in preference", please let me know and I'll make the switch!

@PandaMonkey
Copy link

Hi, @michael-simons , there is a dependency conflict between tika-core 1.17 and 1.12 in your project now. Your project actually depends on tika-core 1.17, and if you plan to exclude the redundant version tika-core 1.12 which is transitivity introduced by vorbis-java-tika, you can add the following patch in your pom. Hope my answer really helps you.

    <dependency>
        <groupId>org.gagravarr</groupId>
        <artifactId>vorbis-java-tika</artifactId>
        <version>0.8</version>
        <exclusions>
            <exclusion>
	    <groupId>org.apache.tika</groupId>
                <artifactId>tika-core</artifactId>                                    
            </exclusion>
        </exclusions>
    </dependency>

@michael-simons
Copy link
Author

Thanks a lot @PandaMonkey That's what I applied. Much appreciated!

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

No branches or pull requests

3 participants