Skip to content

Dependency Updates#3805

Merged
merks merged 1 commit into
masterfrom
update_target
May 6, 2026
Merged

Dependency Updates#3805
merks merged 1 commit into
masterfrom
update_target

Conversation

@eclipse-releng-bot
Copy link
Copy Markdown
Contributor

@eclipse-releng-bot eclipse-releng-bot commented May 5, 2026

The content of the target eclipse-sdk-prereqs.target was updated

Please 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.8 has been updated to version 12.1.9
  • org.eclipse.jetty.ee8:jetty-ee8-nested:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.content [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.pathmap [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server.handler [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.session [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.annotation [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.resource [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty.ee8:jetty-ee8-security:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.ee8.nested [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.pathmap [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.ssl [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty.ee8:jetty-ee8-servlet:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.ee8.nested [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.ee8.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.content [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.pathmap [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server.handler [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-http:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io.content [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.annotation [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.compression [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.resource [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-io:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.annotation [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.resource [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.ssl [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.statistic [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread.strategy [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-security:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.pathmap [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server.handler [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.resource [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.security [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.ssl [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-server:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.content [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.http.pathmap [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io.content [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io.ssl [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.annotation [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.component [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.compression [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.resource [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.ssl [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.statistic [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread.strategy [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-session:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.http [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.io [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.server.handler [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-util-ajax:jar:12.1.8 has been updated to version 12.1.9
    • additionally requires java.package; org.eclipse.jetty.util [12.1.9,13.0.0) compared to the previous version
    • additionally requires java.package; org.eclipse.jetty.util.thread [12.1.9,13.0.0) compared to the previous version
  • org.eclipse.jetty:jetty-util:jar:12.1.8 has been updated to version 12.1.9

@akurtakov
Copy link
Copy Markdown
Member

@HeikoKlare What should we do here? We can not and should not add more dependencies to swt for the sake of logging.

@akurtakov
Copy link
Copy Markdown
Member

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.

@HeikoKlare
Copy link
Copy Markdown
Contributor

With the JSVG bundle, we already have an (optional) dependency to SWT anyway. With version 2.1.0, the equivalent seems to be using com.github.weisj.jsvg.logging.systemlogger additionally, as it contains the logger implementation that was previously contained in the JSVG bundle itself.

As an alternative, we could just have an implementation of the LogManager inside SWT's SVG fragment and provide the according capability, couldn't we?

@akurtakov
Copy link
Copy Markdown
Member

Implementation of the LogManager in the svg fragment is what I would prefer.

@merks
Copy link
Copy Markdown
Contributor

merks commented May 5, 2026

I have this all working such that no additional logging provider is needed

image

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

@eclipse-releng-bot eclipse-releng-bot force-pushed the update_target branch 3 times, most recently from 4c1fe85 to e79cbae Compare May 6, 2026 09:37
@merks
Copy link
Copy Markdown
Contributor

merks commented May 6, 2026

This has now become just a jetty update.

The jsvg update was done by

#3806

The corresponding SWT changes are done in

eclipse-platform/eclipse.platform.swt#3283

@merks
Copy link
Copy Markdown
Contributor

merks commented May 6, 2026

/request-license-review

@merks
Copy link
Copy Markdown
Contributor

merks commented May 6, 2026

FYI, I expect the license review to fail with the duplicate jsvg iplabl issue...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

/request-license-review

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):
https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/actions/runs/25430792098

@merks
Copy link
Copy Markdown
Contributor

merks commented May 6, 2026

Ignoring duplicate jsvg iplab issues.

@merks merks merged commit 51e8b4e into master May 6, 2026
6 of 7 checks passed
@merks merks deleted the update_target branch May 6, 2026 11:54
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