Github user lewismc commented on a diff in the pull request:

    https://github.com/apache/jena/pull/474#discussion_r222383653
  
    --- Diff: jena-jdbc/jena-jdbc-core/pom.xml ---
    @@ -39,17 +39,28 @@
                        <version>3.10.0-SNAPSHOT</version>
                </dependency>
     
    -    <!-- Jena Driver configures Log4j directly -->
    -    <dependency>
    -      <groupId>org.slf4j</groupId>
    -      <artifactId>slf4j-log4j12</artifactId>
    -    </dependency>
    +           <!-- Jena Driver configures Log4j directly -->
    +           <dependency>
    +                   <groupId>org.slf4j</groupId>
    +                   <artifactId>slf4j-log4j12</artifactId>
    +           </dependency>
    +
    +           <dependency>
    +                   <groupId>log4j</groupId>
    +                   <artifactId>log4j</artifactId>
    +           </dependency>
    +
    +           <dependency>
    +                   <groupId>org.apache.logging.log4j</groupId>
    +                   <artifactId>log4j-api</artifactId>
    +           </dependency>
    +
    +           <!-- Intercept direct use of log4j2 -->
    +           <dependency>
    +                   <groupId>org.apache.logging.log4j</groupId>
    --- End diff --
    
    From what I understand, the offending class 
[org.apache.log4j.PropertyConfigurator](https://logging.apache.org/log4j/2.0/log4j-1.2-api/apidocs/index.html?org/apache/log4j/PropertyConfigurator.html)
 resides in log4j 2.0... the runtime dependencies are detailed [as 
follows](https://logging.apache.org/log4j/2.0/runtime-dependencies.html) with 
the [Maven Artifacts detailed 
here](https://logging.apache.org/log4j/2.0/maven-artifacts.html).


---

Reply via email to