[java-opensaml] branch main updated: OSJ-337 The storage-impl POM has compile-time Spring deps

Scott Cantor cantor.2 at osu.edu
Tue Jul 20 16:12:32 UTC 2021


This is an automated email from the git hooks/post-receive script.

scantor pushed a commit to branch main
in repository java-opensaml.

View the commit online:
http://git.shibboleth.net/view/?p=java-opensaml.git;a=commit;h=b89e8b4fb528d09af949f749c73c8fd42ee65734

The following commit(s) were added to refs/heads/main by this push:
       new  b89e8b4fb OSJ-337 The storage-impl POM has compile-time Spring deps
b89e8b4fb is described below

commit b89e8b4fb528d09af949f749c73c8fd42ee65734
Author: Scott Cantor <cantor.2 at osu.edu>
AuthorDate: Tue Jul 20 12:12:29 2021 -0400

    OSJ-337 The storage-impl POM has compile-time Spring deps
    
    https://issues.shibboleth.net/jira/browse/OSJ-337
---
 opensaml-storage-impl/pom.xml | 38 ++++++++++++++++++++++++--------------
 1 file changed, 24 insertions(+), 14 deletions(-)

diff --git a/opensaml-storage-impl/pom.xml b/opensaml-storage-impl/pom.xml
index 69b866e09..e26fbd928 100644
--- a/opensaml-storage-impl/pom.xml
+++ b/opensaml-storage-impl/pom.xml
@@ -50,24 +50,18 @@
         	<artifactId>guava</artifactId>
         </dependency>
 
+        <!-- Needed for LDAP storage plugin. -->
         <dependency>
             <groupId>org.ldaptive</groupId>
             <artifactId>ldaptive</artifactId>
+            <optional>true</optional>
         </dependency>
 
-		<dependency>
-			<groupId>org.springframework</groupId>
-			<artifactId>spring-beans</artifactId>
-		</dependency>
-
-		<dependency>
-			<groupId>org.springframework</groupId>
-			<artifactId>spring-core</artifactId>
-		</dependency>
-
+        <!-- Needed for Memcache storage plugin. -->
         <dependency>
             <groupId>net.spy</groupId>
             <artifactId>spymemcached</artifactId>
+            <optional>true</optional>
         </dependency>
 
         <dependency>
@@ -75,9 +69,11 @@
             <artifactId>cryptacular</artifactId>
         </dependency>
 
+        <!-- Needed for JPA storage plugin. -->
 		<dependency>
 			<groupId>org.hibernate</groupId>
 			<artifactId>hibernate-core</artifactId>
+            <optional>true</optional>
 		</dependency>
 
         <!-- Provided Dependencies -->
@@ -97,9 +93,14 @@
             <artifactId>jakarta.json</artifactId>
             <scope>runtime</scope>
         </dependency>
+        
+        <!-- Needed for JPA storage plugin. -->
+        <!-- TODO: I think is test scope here but runtime for the IdP. -->
         <dependency>
             <groupId>${spring.groupId}</groupId>
             <artifactId>spring-orm</artifactId>
+            <scope>runtime</scope>
+            <optional>true</optional>
         </dependency>
 
         <!-- Test Dependencies -->
@@ -151,20 +152,29 @@
             <scope>test</scope>
         </dependency>
 
+        <dependency>
+            <groupId>org.springframework</groupId>
+            <artifactId>spring-beans</artifactId>
+            <scope>test</scope>
+        </dependency>
         <dependency>
             <groupId>${spring.groupId}</groupId>
-            <artifactId>spring-test</artifactId>
+            <artifactId>spring-context</artifactId>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>org.springframework</groupId>
+            <artifactId>spring-core</artifactId>
             <scope>test</scope>
         </dependency>
         <dependency>
             <groupId>${spring.groupId}</groupId>
-            <artifactId>spring-web</artifactId>
+            <artifactId>spring-test</artifactId>
             <scope>test</scope>
         </dependency>
-
         <dependency>
             <groupId>${spring.groupId}</groupId>
-            <artifactId>spring-context</artifactId>
+            <artifactId>spring-web</artifactId>
             <scope>test</scope>
         </dependency>
 

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list