Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ISSUE #4700] fix: exclude spring-boot-starter-logging #4861

Closed
wants to merge 1 commit into from

Conversation

fabian4
Copy link
Member

@fabian4 fabian4 commented Apr 24, 2024

Fixes #4700

Motivation

Run org.apache.eventmesh.tcp.demo.pub.cloudevents.SyncRequest Failed

> Task :eventmesh-examples:SyncRequest.main() FAILED
22 actionable tasks: 1 executed, 1 from cache, 20 up-to-date
Exception in thread "main" java.lang.ExceptionInInitializerError
Caused by: org.apache.logging.log4j.LoggingException: log4j-slf4j2-impl cannot be present with log4j-to-slf4j
	at org.apache.logging.slf4j.Log4jLoggerFactory.validateContext(Log4jLoggerFactory.java:67)
	at org.apache.logging.slf4j.Log4jLoggerFactory.newLogger(Log4jLoggerFactory.java:49)
	at org.apache.logging.slf4j.Log4jLoggerFactory.newLogger(Log4jLoggerFactory.java:32)
	at org.apache.logging.log4j.spi.AbstractLoggerAdapter.getLogger(AbstractLoggerAdapter.java:52)
	at org.apache.logging.slf4j.Log4jLoggerFactory.getLogger(Log4jLoggerFactory.java:32)
	at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:426)
	at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:451)
	at org.apache.eventmesh.tcp.demo.pub.cloudevents.SyncRequest.<clinit>(SyncRequest.java:41)

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':eventmesh-examples:SyncRequest.main()'.

Modifications

exclude spring-boot-starter-logging in eventmesh-examples/build.gradle

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@Pil0tXia
Copy link
Member

Pil0tXia commented Apr 24, 2024

This change is included in be06ef7 of #4831 but haven't merged yet.

#4831 is ready for merge. May you please help review?

@Pil0tXia
Copy link
Member

Please check current master branch~ This issue should have been fixed.

@fabian4 fabian4 closed this Apr 26, 2024
@fabian4 fabian4 deleted the fix-log4j-slf4j branch May 6, 2024 08:44
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.

[Bug] Maven artifacts should not depend on logging backend
2 participants