Bug 7285 - CENTINEL Carbon Autorelease Build Failure

guice dependency shouldn't be targeted with scope test
as downstream consumer of that dependency needs it during
some other lifecycle state (compile).

Centinel is pulling graylog2-plugin that requires guice at
compile time, and does pull it as transitice dependency.
But since odlparent provide it with the scope test, and centinel
isn't speficically defining that dependency, odlparent overwrite
the transitive dependency of graylog2-plugin

Change-Id: I9c109f942804637463113db56d0bce45d4cf65ae
Signed-off-by: Alexis de Talhouët <adetalhouet@inocybe.com>

diff --git a/odlparent/pom.xml b/odlparent/pom.xml
index c8a3927ebf7ab7b7747c9c0c9c65b7474dc310e7..235413187a4491f39cc9ce5804568d4c7c5f9e4f 100644 (file)
--- a/odlparent/pom.xml
+++ b/odlparent/pom.xml
@@ -305,7 +305,6 @@
       </dependency>
       <dependency>
         <groupId>com.mycila.guice.extensions</groupId>
-        <!-- This transitively pulls com.google.inject:guice along -->
         <artifactId>mycila-guice-jsr250</artifactId>
         <!-- Make sure this version is in sync with the guice one below -->
         <version>4.0.rc1</version>
@@ -319,13 +318,11 @@
         <scope>provided</scope>
       </dependency>
       <dependency>
-        <!-- We repeat guice here, because some POM need only Guice, without mycila -->
         <groupId>com.google.inject</groupId>
         <artifactId>guice</artifactId>
         <!-- Make sure this version is in sync with the one used by mycila-guice-jsr250 above
              by checking it here: https://github.com/mycila/guice/blob/master/pom.xml#L103 -->
         <version>4.1.0</version>
-        <scope>test</scope>
       </dependency>
       <dependency>
         <groupId>org.apache.commons</groupId>