Some minor version/scope cleanups. Hamcrest should be a test dependency. Make sure we pull in Log4j2 2.17.0. In QSARCML log4j-core should only be in the tests

johnmay · egonw · commit 268e172e8a6e · 2021-12-22T18:19:26.000+01:00
diff --git a/descriptor/qsarcml/pom.xml b/descriptor/qsarcml/pom.xml
@@ -36,6 +36,7 @@
         <dependency>
             <groupId>org.apache.logging.log4j</groupId>
             <artifactId>log4j-core</artifactId>
+            <scope>test</scope>
         </dependency>
         <dependency>
             <groupId>junit</groupId>
diff --git a/pom.xml b/pom.xml
@@ -509,7 +509,12 @@
             <dependency>
                 <groupId>org.apache.logging.log4j</groupId>
                 <artifactId>log4j-1.2-api</artifactId>
-                <version>2.16.0</version>
+                <version>2.17.0</version>
+            </dependency>
+            <dependency>
+                <groupId>org.apache.logging.log4j</groupId>
+                <artifactId>log4j-api</artifactId>
+                <version>2.17.0</version>
             </dependency>
             <dependency>
                 <groupId>org.apache.logging.log4j</groupId>
diff --git a/storage/smiles/pom.xml b/storage/smiles/pom.xml
@@ -30,9 +30,10 @@
             <groupId>uk.ac.ebi.beam</groupId>
             <artifactId>beam-func</artifactId>
         </dependency>
-            <dependency>
+        <dependency>
             <groupId>org.hamcrest</groupId>
             <artifactId>hamcrest</artifactId>
+            <scope>test</scope>
         </dependency>
         <dependency>
             <groupId>junit</groupId>
