[GitHub] jena pull request: JENA-583: Using Maven optional tag for log4j de...

2016-02-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jena/pull/117 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] jena pull request: JENA-583: Using Maven optional tag for log4j de...

2016-01-01 Thread afs
Github user afs commented on the pull request: https://github.com/apache/jena/pull/117#issuecomment-168302946 The codebase should compile with a clean maven repo. Jena does not compile without this dependency. This is a discussion needed -- see JENA-1005. --- If your

[GitHub] jena pull request: JENA-583: Using Maven optional tag for log4j de...

2016-01-01 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/pull/117#issuecomment-168308691 This PR builds with `mvn clean install` or, to be extra-sure: ``` mvn clean dependency:purge-local-repository -DreResolve=false mvn install ``` --- If your

[GitHub] jena pull request: JENA-583: Using Maven optional tag for log4j de...

2016-01-01 Thread ajs6f
Github user ajs6f commented on the pull request: https://github.com/apache/jena/pull/117#issuecomment-168305858 Having the [optional tag](https://maven.apache.org/guides/introduction/introduction-to-optional-and-excludes-dependencies.html) doesn't prevent Jena from using this

[GitHub] jena pull request: JENA-583: Using Maven optional tag for log4j de...

2015-12-31 Thread ajs6f
GitHub user ajs6f opened a pull request: https://github.com/apache/jena/pull/117 JENA-583: Using Maven optional tag for log4j dependencies You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajs6f/jena JENA-583 Alternatively you