Commit ac1d0cab authored by Andy Wilkinson's avatar Andy Wilkinson

Prevent Gradle from pulling in groovy-all with the remote shell starter

Sadly, Gradle handle's exclusions differently to Maven even when it's
processing a Maven pom.

In this case groovy-all is pulled in via org.crashub:crash.shell where
we've excluded it. This is enough to prevent Maven from pulling in
groovy-all when you depend on the remote shell starter.
org.crashub:crash.shell is also pulled in as a transitive dependency
of a number of other dependencies and Gradle requires each of these
to also exclude groovy-all for it to actually be excluded.

This commit adds the additional exclusions that are required to make
Gradle's behaviour sane.

Fixes gh-2257
parent bd83aca6
...@@ -47,6 +47,12 @@ ...@@ -47,6 +47,12 @@
<dependency> <dependency>
<groupId>org.crashub</groupId> <groupId>org.crashub</groupId>
<artifactId>crash.connectors.ssh</artifactId> <artifactId>crash.connectors.ssh</artifactId>
<exclusions>
<exclusion>
<groupId>org.codehaus.groovy</groupId>
<artifactId>groovy-all</artifactId>
</exclusion>
</exclusions>
</dependency> </dependency>
<dependency> <dependency>
<groupId>org.crashub</groupId> <groupId>org.crashub</groupId>
...@@ -75,15 +81,31 @@ ...@@ -75,15 +81,31 @@
<artifactId>spring-web</artifactId> <artifactId>spring-web</artifactId>
<groupId>org.springframework</groupId> <groupId>org.springframework</groupId>
</exclusion> </exclusion>
<exclusion>
<groupId>org.codehaus.groovy</groupId>
<artifactId>groovy-all</artifactId>
</exclusion>
</exclusions> </exclusions>
</dependency> </dependency>
<dependency> <dependency>
<groupId>org.crashub</groupId> <groupId>org.crashub</groupId>
<artifactId>crash.plugins.cron</artifactId> <artifactId>crash.plugins.cron</artifactId>
<exclusions>
<exclusion>
<groupId>org.codehaus.groovy</groupId>
<artifactId>groovy-all</artifactId>
</exclusion>
</exclusions>
</dependency> </dependency>
<dependency> <dependency>
<groupId>org.crashub</groupId> <groupId>org.crashub</groupId>
<artifactId>crash.plugins.mail</artifactId> <artifactId>crash.plugins.mail</artifactId>
<exclusions>
<exclusion>
<groupId>org.codehaus.groovy</groupId>
<artifactId>groovy-all</artifactId>
</exclusion>
</exclusions>
</dependency> </dependency>
<dependency> <dependency>
<groupId>org.crashub</groupId> <groupId>org.crashub</groupId>
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment