Some minor version/scope cleanups. Hamcrest should be a test dependen…

…cy. Make sure we pull in Log4j2 2.17.0. In QSARCML log4j-core should only be in the tests
cdk · Dec 22, 2021 · 268e172 · 268e172
1 parent 0e24cbc
commit 268e172
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
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>