[
https://issues.apache.org/jira/browse/JENA-1609?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16636343#comment-16636343
]
ASF GitHub Bot commented on JENA-1609:
--------------------------------------
Github user lewismc commented on a diff in the pull request:
https://github.com/apache/jena/pull/474#discussion_r222164161
--- 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>
--- End diff --
Ok thanks @rvesse. The logic seems clear from what you wrote, it doesn't
however take away from the fact that the codebase will not build under JDK10
(and possibly JDK9) without the presence of the log4j library containing
org.apache.log4j.PropertyConfigurator as currently used within
jena/jena-jdbc/jena-jdbc-core/src/main/java/org/apache/jena/jdbc/JenaDriver.java.
The issue here is to either refactor the logging or to add the
dependency... I would think there is no other scenario under consideration.
Is that a fair statement?
> Jena master does not build under JDK10
> --------------------------------------
>
> Key: JENA-1609
> URL: https://issues.apache.org/jira/browse/JENA-1609
> Project: Apache Jena
> Issue Type: Improvement
> Components: Build, Documentation
> Environment: Apache Maven 3.5.4
> (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T11:33:14-07:00)
> Maven home: /usr/local/Cellar/maven/3.5.4/libexec
> Java version: 10.0.2, vendor: Oracle Corporation, runtime:
> /Library/Java/JavaVirtualMachines/jdk-10.0.2.jdk/Contents/Home
> Default locale: en_US, platform encoding: UTF-8
> OS name: "mac os x", version: "10.12.6", arch: "x86_64", family: "mac"
> Reporter: Lewis John McGibbney
> Priority: Major
> Fix For: Jena 3.9.0
>
>
> When I attempt build under Oracle JDK 10 there are several issues which cause
> the build to fail. The following PR addresses these issues and enables me to
> build and run Jena master under JDK 10.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
