Skip to content

ARTEMIS-3453: exclude log4j transitive dep from zookeeper#3712

Merged
asfgit merged 1 commit intoapache:mainfrom
gemmellr:zookeeper-log4j
Sep 1, 2021
Merged

ARTEMIS-3453: exclude log4j transitive dep from zookeeper#3712
asfgit merged 1 commit intoapache:mainfrom
gemmellr:zookeeper-log4j

Conversation

@gemmellr
Copy link
Member

https://issues.apache.org/jira/browse/ARTEMIS-3453

Didnt test this at all yet other than seeing its effect on the main build. Seems the way to go though so raising for a PR run and any input from others.

In particular, by excluding sl4fj-log4j and log4j across the board, it may be necessary to add other dependencies to some modules (examples?) if they lack a another means of output such as another slf4j bridge.

@gtully
Copy link
Contributor

gtully commented Aug 31, 2021

+1 it is the way to go

@franz1981
Copy link
Contributor

Will give a shot to see what can happen here

@gemmellr
Copy link
Member Author

gemmellr commented Sep 1, 2021

I dont see any examples depending on zookeeper directly, so I'm going to push the change as-is, can always follow up if needed.

@asfgit asfgit merged commit 359b919 into apache:main Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants