Dependency Updates#3805
Conversation
|
@HeikoKlare What should we do here? We can not and should not add more dependencies to swt for the sake of logging. |
|
From what I understand one needs either https://repo1.maven.org/maven2/com/github/weisj/jsvg-slf4j/ or https://repo1.maven.org/maven2/com/github/weisj/jsvg-systemlogger/ on the classpath to use jsvg - but this is a clear NOGO for SWT IMO. |
|
With the JSVG bundle, we already have an (optional) dependency to SWT anyway. With version 2.1.0, the equivalent seems to be using As an alternative, we could just have an implementation of the |
|
Implementation of the LogManager in the svg fragment is what I would prefer. |
|
I have this all working such that no additional logging provider is needed
The implementation details in jsvg are significantly bogus. We really need to set the context class loader so it loads are service. That makes the whole OSGI provided/require capability stuff kind of bogus. When running standalone it doesn't load the swt.svg-provided service but rather the default. In the end, the service requirement should have been declared optional because one is not required. This review is pending: https://gitlab.eclipse.org/eclipsefdn/emo-team/iplab/-/issues/27907 |
4c1fe85 to
e79cbae
Compare
e79cbae to
bfa3552
Compare
|
This has now become just a jetty update. The jsvg update was done by The corresponding SWT changes are done in |
|
/request-license-review |
|
FYI, I expect the license review to fail with the duplicate jsvg iplabl issue... |
License review requests:
After all reviews have concluded, re-run the license-vetting check from the Github Actions web-interface to update its status. Workflow run (with attached summary files): |
|
Ignoring duplicate jsvg iplab issues. |

The content of the target
eclipse-sdk-prereqs.targetwas updatedPlease review the changes and merge if appropriate, or cherry pick individual updates.
The following maven artifacts have been updated:
org.eclipse.jetty.ee8:jetty-ee8-apache-jsp:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty.ee8:jetty-ee8-nested:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty.ee8:jetty-ee8-security:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty.ee8:jetty-ee8-servlet:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-http:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-io:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-security:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-server:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-session:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-util-ajax:jar:12.1.8has been updated to version12.1.9org.eclipse.jetty:jetty-util:jar:12.1.8has been updated to version12.1.9