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

[Suggestion] Support library #405

Closed
j-garin opened this issue Oct 2, 2017 · 4 comments
Closed

[Suggestion] Support library #405

j-garin opened this issue Oct 2, 2017 · 4 comments

Comments

@j-garin
Copy link

j-garin commented Oct 2, 2017

Hi. I have a suggestion for you. Could you please use
implementation 'com.android.support:exifinterface:<your version>
or

compile ('com.android.support:exifinterface:<your version>) {
    transitive = false
}```
to avoid support library version conflicts in following versions?  Thank you.
@ArthurHub
Copy link
Owner

Can you link some documentation explaining what it does?

@lisawray
Copy link

lisawray commented Oct 3, 2017

I suspect this is related:

Error:Execution failed for task ':app:processDebugManifest'.
Manifest merger failed : Attribute meta-data#android.support.VERSION@value value=(26.1.0) from [com.android.support:design:26.1.0] AndroidManifest.xml:28:13-35
is also present at [com.android.support:exifinterface:26.0.2] AndroidManifest.xml:25:13-35 value=(26.0.2).
Suggestion: add 'tools:replace="android:value"' to element at AndroidManifest.xml:26:9-28:38 to override.

(Edited from my previous solution)
To fix it, I have to put in my own build.gradle:

configurations.all {
    resolutionStrategy {
        force "com.android.support:exifinterface:$supportLibVersion"
    }
}

I'm no gradle expert but I hope that helps understand what's going on!

@j-garin
Copy link
Author

j-garin commented Oct 4, 2017

@lisawray exactly
@ArthurHub transitive libraries will be available in modules, which depend on your library. This can cause issues when people use other library versions than you. @lisawray gave you a good example just above. If you set transitive to false, such conflicts will no longer occur even without that workaround.
More info on that here: official gradle docs

@ArthurHub
Copy link
Owner

added in 2.5.1, hope it solves the merge issue, thx @j-garin

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