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

Fixes #962 by disabling the deegree-maven-plugin goal assemble-log4j #965

Merged
merged 1 commit into from
Mar 29, 2019

Conversation

tfr42
Copy link
Member

@tfr42 tfr42 commented Mar 28, 2019

Fixes #962 by disabling the deegree-maven-plugin goal assemble-log4j and provides a default log4j.properties configuration file matching current log4j 1.2 provider.
This is considered as a temporary fix.

…-log4j and provides a default log4j.properties configuration file matching current log4j 1.2 provider
@copierrj copierrj merged commit 7a80212 into deegree:master Mar 29, 2019
@deegree-ci deegree-ci added the bug error issue and bug (fix) label Mar 30, 2019
@deegree-ci deegree-ci added this to the 3.4.5 milestone Mar 30, 2019
@tfr42
Copy link
Member Author

tfr42 commented Apr 2, 2019

Excerpt from TMC meeting 20190329:
The TMC has the motion to simplify the build by removing the maven plugin for generating the configuration file and keep a static log4j configuration. The file format properties shall be kept as well. But an update to the most recent version of Apache log4j2 is appreciated.

@tfr42 tfr42 mentioned this pull request Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug error issue and bug (fix)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Log4j configuration is invalid in deegree 3.4.4
3 participants