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

make plugin work for multi-module android project with flatDirs #59

Merged
merged 1 commit into from
May 27, 2015
Merged

make plugin work for multi-module android project with flatDirs #59

merged 1 commit into from
May 27, 2015

Conversation

dbachelder
Copy link
Contributor

some quick fixes to get things working for our project... not sure if you want these changes or not...

@ben-manes
Copy link
Owner

Thanks! What are the benefits between this approach and ignoring flatDir repositories? I can think of two possible ones.

First that when dependency was unresolved, due to ignoring the repository, there was no exceptional cause and thus no clear reason given in the log. That may have been partially due to the logging bug you fixed.

The other is that even though flatDir repositories lack metadata, the version may be inferred by the dependency name. This may make the dependency comparable against a version found in an external repository and provide meaningful update information. I'm not sure if this is what Gradle will actually due, though.

@dbachelder
Copy link
Contributor Author

I was still ending up with NPE's when run against our project. I think hiding the flatDir repos wasn't actually hiding the dependencies.

@jochenberger jochenberger added this to the 0.10 milestone May 27, 2015
@ben-manes ben-manes merged commit d882701 into ben-manes:master May 27, 2015
ben-manes added a commit that referenced this pull request May 27, 2015
The test fails with my fix for #36, which passed the provided example
project, due to an NPE using the comparator. The test passes with the
fix provided by #59, validating @dbachelder's solution.
@ben-manes
Copy link
Owner

Thanks! I wrote a unit test that reproduced the NPE, which your changes fix.

@jochenberger Please perform a release when you have the time.

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

Successfully merging this pull request may close these issues.

None yet

3 participants