Skip to content
This repository was archived by the owner on Apr 19, 2018. It is now read-only.

Conversation

plastiv
Copy link

@plastiv plastiv commented Nov 18, 2015

Perfoms dependency tree scan to find if any of the top level
dependencies listed at the build.gradle file depends on support
library and though android m2repository needs to be downloaded.

NOTE: establishing network connection and downloading dependency pom
are required to analyze if dependency has childrens.

Implementation is inspired by official gradle "dependency" task
which prints dependency tree
https://github.com/gradle/gradle/blob/master/subprojects/diagnostics/src/main/groovy/org/gradle/api/tasks/diagnostics/AbstractDependencyReportTask.java#L58

Perfoms dependency tree scan to find if any of the top level
dependencies listed at the build.gradle file depends on support
library and though android m2repository needs to be downloaded.

NOTE: establishing network connection and downloading dependency pom
are required to analyze if dependency has childrens.

Implementation is inspired by official gradle "dependency" task
which prints dependency tree
https://github.com/gradle/gradle/blob/master/subprojects/diagnostics/src/main/groovy/org/gradle/api/tasks/diagnostics/AbstractDependencyReportTask.java#L58
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have moved adding android repository url above dependency scan routine.
This is necessary because now we are not just parsing strings at the build.gradle file, but gradle actually tries to resolve dependencies. And to be able to resolve the "up-to-date" case gradle needs m2repository link to resolve support library if it's repository is present.

@plastiv
Copy link
Author

plastiv commented Nov 18, 2015

Same thing could be done for play-services check

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant