> @@ -306,6 +308,16 @@
> <version>${assertj-guava.version}</version>
> </dependency>
> <dependency>
> + <groupId>com.google.auto.factory</groupId>
> + <artifactId>auto-factory</artifactId>
> + <version>${auto-factory.version}</version>
> + </dependency>
> + <dependency>
> + <groupId>com.google.auto.service</groupId>
> I'd rather not add it by default
TL;DR: +1 to that. Unless we get feedback that copy-pasting a couple of deps is
so annoying that we need to avoid the need for it, I'd rather keep our user
impact down by keeping the list of transitive deps as small as possible.
The set of dependencies that we propagate to all modules that inherit from
jclouds-project should be as small as possible. I agree with @jdaggett that we
want to use `Auto*` wherever possible, but putting the deps in `<dependency>`
doesn't enforce that anyway, and copy-pasting a couple of deps when creating a
POM doesn't seem like a huge overhead at this point (certainly not when
compared to the overall size of our POMs...but that's another discussion ;-) )
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/609/files#r20896378
