Yep, this is GEODE-26.

Anthony

> On Feb 8, 2016, at 7:24 PM, John Blum <[email protected]> wrote:
> 
> mavenLocal() should definitely *not* be in the repositories { ... }
> declaration
> <https://github.com/apache/incubator-geode/blob/rel/v1.0.0-incubating.M1/build.gradle#L47-L48>
> [0]
> of build.gradle.
> 
> I am actually quite surprised this passed either the Apache or Sonatype's
> release process.  Sonatype is usually quite strict about listing any
> dependencies in you POM that cannot be resolved from Maven Central.
> 
> The problem with declaring mavenLocal() in build.gradle (as I mentioned
> before) is that a developer could inadvertently use a dependency or version
> of a dependency that is not available in Maven Central, thus causing the
> resolution of transitive dependencies to fail when someone is using Apache
> Geode for development.
> 
> It is a simple matter to add mavenLocal() to your personal
> ~/.gradle/init.gradle file if necessary...
> 
> allprojects {
>  repositories {
>    mavenLocal()
>    maven { url "http://dist.gemstone.com/maven/release"}
>    ...
>  }
> }
> 
> By default, Gradle, like Maven, should pull automatically from Maven
> Central, thus making the explicit mavenCentral() declaration unnecessary.
> 
> Cheers,
> John
> 
> [0] -
> https://github.com/apache/incubator-geode/blob/rel/v1.0.0-incubating.M1/build.gradle#L47-L48

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to