You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "upthewaterspout (GitHub)" <gi...@apache.org> on 2019/09/20 19:04:26 UTC

[GitHub] [geode] upthewaterspout commented on issue #4075: GEODE-6964: Move geode log4j core classes to geode-log4j

> If `LogManager` is still in `geode-core` then `log4j-api` is an `api` scoped module. Downstream modules that use `LogManager` need `log4j-api` on the compile class path, which is what `api` scope will do.

log4j-api used to be an implementation dependency. I think @kirklund originally flipped it to API because he was adding new public API that depended on log4j-api, and didn't flip it back to implementation when he made that API internal again.

[ Full content available at: https://github.com/apache/geode/pull/4075 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org