You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2016/09/05 12:29:42 UTC

[06/10] jena git commit: Exclude slf4j-jul bridge.

Exclude slf4j-jul bridge.

The dependency jena-fuseki-core may change.
This protects against that.
Expand comment.
Remove example logging dependencies.


Project: http://git-wip-us.apache.org/repos/asf/jena/repo
Commit: http://git-wip-us.apache.org/repos/asf/jena/commit/1d744688
Tree: http://git-wip-us.apache.org/repos/asf/jena/tree/1d744688
Diff: http://git-wip-us.apache.org/repos/asf/jena/diff/1d744688

Branch: refs/heads/master
Commit: 1d7446882575851d840d2b5ea285d38573638c9c
Parents: 26958bb
Author: Andy Seaborne <an...@apache.org>
Authored: Sun Sep 4 18:34:09 2016 +0100
Committer: Andy Seaborne <an...@apache.org>
Committed: Sun Sep 4 18:34:09 2016 +0100

----------------------------------------------------------------------
 jena-fuseki2/jena-fuseki-embedded/pom.xml | 41 ++++++++------------------
 1 file changed, 13 insertions(+), 28 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jena/blob/1d744688/jena-fuseki2/jena-fuseki-embedded/pom.xml
----------------------------------------------------------------------
diff --git a/jena-fuseki2/jena-fuseki-embedded/pom.xml b/jena-fuseki2/jena-fuseki-embedded/pom.xml
index 58429a1..4bd4e99 100644
--- a/jena-fuseki2/jena-fuseki-embedded/pom.xml
+++ b/jena-fuseki2/jena-fuseki-embedded/pom.xml
@@ -39,8 +39,10 @@
       <version>2.4.1-SNAPSHOT</version>
       <!-- No specific logging - leave to the application -->
       <exclusions>
-
-        <!-- Parts of Jena not needed but picked up via jena-fuseki-core -->
+        <!-- Parts of Jena not needed but picked up via jena-fuseki-core.
+             The application can put back any of these if it wants to use
+             them, e.g. a text-indexed dataset, providing the version is compatible.
+	    -->
         <exclusion>
           <groupId>org.apache.jena</groupId>
           <artifactId>jena-cmds</artifactId>
@@ -53,8 +55,11 @@
           <groupId>org.apache.jena</groupId>
           <artifactId>jena-spatial</artifactId>
         </exclusion>
-        
-        <!-- Exclude choice of logging. Application responsibility. -->
+        <!-- Jena use SLF4J as its API. Exclude any logging bindings picked up via Jena.
+          For example, jena-fuseki-core has logging output and so some logging provider.
+          For embedded use, it is the enclosing application that chooses the kind of
+          logging system.    
+        --> 
         <exclusion>
           <groupId>org.slf4j</groupId>
           <artifactId>slf4j-log4j12</artifactId>
@@ -63,33 +68,13 @@
           <groupId>log4j</groupId>
           <artifactId>log4j</artifactId>
         </exclusion>
-
+        <exclusion>
+          <groupId>org.slf4j</groupId>
+          <artifactId>slf4j-jdk14</artifactId>
+        </exclusion>
       </exclusions>
     </dependency>
 
-    <!-- Logging examples -->
-    <!-- java.util.logging.
-    <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-jdk14</artifactId>
-      <version>1.7.21</version>
-    </dependency>
-    -->
-
-    <!-- Log4j : need 2 dependencies.
-    <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-log4j12</artifactId>
-      <version>1.7.21</version>
-    </dependency>
-
-    <dependency>
-      <groupId>log4j</groupId>
-      <artifactId>log4j</artifactId>
-      <version>1.2.17</version>
-    </dependency> 
-    -->
-
   </dependencies>
 
   <build>