[geode] 01/02: GEODE-7058: Mark log4j-core optional in geode-core

[on...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

onichols pushed a commit to branch release/1.10.0
in repository https://gitbox.apache.org/repos/asf/geode.git

commit 1d5bbe1fcd6d0f2043a6406a24a817febc94908c
Author: Kirk Lund <kl...@apache.org>
AuthorDate: Wed Aug 7 14:33:21 2019 -0700

    GEODE-7058: Mark log4j-core optional in geode-core
    
    Note: this change requires all commits from GEODE-2644 and GEODE-6122.
    (cherry picked from commit 413800bc16d05df689a2af5c30797f180aad6088)
---
 geode-core/build.gradle                        | 4 +++-
 geode-core/src/test/resources/expected-pom.xml | 1 +
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/geode-core/build.gradle b/geode-core/build.gradle
index cd75837..22b7a77 100755
--- a/geode-core/build.gradle
+++ b/geode-core/build.gradle
@@ -260,7 +260,9 @@ dependencies {
 
   //Log4j is used everywhere
   implementation('org.apache.logging.log4j:log4j-api')
-  implementation('org.apache.logging.log4j:log4j-core')
+  implementation('org.apache.logging.log4j:log4j-core') {
+    ext.optional = true
+  }
 
   //Jansi is used by the CLI (maybe? it is a runtime dependency)
   runtimeOnly('org.fusesource.jansi:jansi') {
diff --git a/geode-core/src/test/resources/expected-pom.xml b/geode-core/src/test/resources/expected-pom.xml
index 8e9618f..ff85364 100644
--- a/geode-core/src/test/resources/expected-pom.xml
+++ b/geode-core/src/test/resources/expected-pom.xml
@@ -224,6 +224,7 @@
       <groupId>org.apache.logging.log4j</groupId>
       <artifactId>log4j-core</artifactId>
       <scope>runtime</scope>
+      <optional>true</optional>
     </dependency>
     <dependency>
       <groupId>org.eclipse.jetty</groupId>