Add Google Guice to dependencyManagement 88/48288/3
authorMichael Vorburger <vorburger@redhat.com>
Sat, 12 Nov 2016 00:55:35 +0000 (01:55 +0100)
committerMichael Vorburger <vorburger@redhat.com>
Thu, 17 Nov 2016 12:42:41 +0000 (13:42 +0100)
Actually we already have it anyway, through mycila, but in
https://git.opendaylight.org/gerrit/#/c/48077/ for
https://git.opendaylight.org/gerrit/#/c/48078/ I'd like to depend only
on Guice directly, because there I don't need anything from mycila, so I
need the adjustment proposed here.

Change-Id: Ia1fcc56c3e650759dc483c8d8c36091e047efde7
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
odlparent/pom.xml

index 84062425034c64f6f177037ef0cac42c66000cdb..d3ed8cd92ba27532a3092528f8af84132c01e48f 100644 (file)
         <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>
         <scope>test</scope>
       </dependency>
         <!-- Must be provided; <scope>test here breaks APT in Eclipse :( -->
         <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>
         <artifactId>commons-lang3</artifactId>