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

Remove JUL over SLF4J and JCL over SLF4J as OMERO server dependencies #6418

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Feb 3, 2025

Fixes #6416

Depends on ome/omero-model#108, ome/omero-server#185 and ome/omero-blitz#155 being integrated in order to unify all OMERO.server components to use the SLF4J API.

The low-level server components have been adjusted to use the slf4j
API directly and no longer require the bridge
@sbesson
Copy link
Member Author

sbesson commented Feb 6, 2025

This PR should now be deployed alongside all associated components PR in the nightly OME CI builds and is ready for functional testing.

At the distribution level, the server bundle should no longer shipe the jul-to-slf4j.jar and jcl-over-slf4j.jar JARs under lib/server. Functionally, there should be no impact to this change. In terms of logging, theJvmSettingsCheck logging statements about the different versions should still be recorded in Blitz-0.log at server startup.

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.

Review and replace usages of JCL/JUL across server components
1 participant