[GitHub] geode issue #626: GEODE-3179: Fixing bundled jars test finding slf4j-jdk jar

2017-07-12 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/geode/pull/626 +1 I'll merge this. --- 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 wishe

[GitHub] geode issue #626: GEODE-3179: Fixing bundled jars test finding slf4j-jdk jar

2017-07-12 Thread DivineEnder
Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/626 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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 proj

Re: slf4j-jdk

2017-07-11 Thread Kirk Lund
Thanks David! Searching git history for slf4j-jdk wasn't yielding much for me. On Mon, Jul 10, 2017 at 4:03 PM, David Anuta wrote: > It looks like that might have been caused by my commit for GEODE-2901. > slf4j-jdk.jar is included in the Tomcat zip folder located within the >

Re: slf4j-jdk

2017-07-10 Thread David Anuta
lt;https://issues.apache.org/jira/browse/GEODE-3179> to resolve the issue and will work on fixing it shortly. On Mon, Jul 10, 2017 at 10:53 AM, Kirk Lund wrote: > Does anyone know how and where "slf4j-jdk" was added as a dependency and > bundled jar? > > I get the feeling that everyo

slf4j-jdk

2017-07-10 Thread Kirk Lund
Does anyone know how and where "slf4j-jdk" was added as a dependency and bundled jar? I get the feeling that everyone is expecting someone else (the person who added it?) is going to fix it. Was it added in a PR maybe? Error java.lang.AssertionError: The bundled jars have changed. P