RFR: 8155 eclipse 2023 12 upgrade

Alex Macdonald aptmac at openjdk.org
Thu Jan 18 19:16:26 UTC 2024


On Thu, 18 Jan 2024 17:50:13 GMT, Virag Purnam <vpurnam at openjdk.org> wrote:

> Eclipse platform upgrade to 2023-12.
> 
> - javax.inject has been replaced with jakarta.inject.
> - added missing features to target file.
> - upgraded tycho to 3.0.5.
> - upgraded jetty to 10.0.18.

Looks okay to me, just a couple of license headers that could be fixed up. 

There's also an Integration Blocker warning: `Title mismatch between PR and JBS for issue`, when you get the chance the PR title can be updated to match the jira issue.

releng/platform-definitions/platform-definition-2023-12/platform-definition-2023-12.target line 3:

> 1: <?xml version="1.0" encoding="UTF-8" standalone="no"?>
> 2: <!--
> 3:    Copyright (c) 2023, 2024, Oracle and/or its affiliates. All rights reserved.

Doesn't need to start with the 2023, also you've added the Datadog licensing line in below as a heads up.

releng/platform-definitions/platform-definition-2023-12/pom.xml line 3:

> 1: <?xml version="1.0" encoding="UTF-8"?>
> 2: <!--
> 3:    Copyright (c) 2023, 2024, Oracle and/or its affiliates. All rights reserved.

Same as the platform definition, doesn't need the 2023 and you brought in the Datadog line as a heads up.

-------------

PR Review: https://git.openjdk.org/jmc/pull/546#pullrequestreview-1830391471
PR Review Comment: https://git.openjdk.org/jmc/pull/546#discussion_r1457875169
PR Review Comment: https://git.openjdk.org/jmc/pull/546#discussion_r1457875804


More information about the jmc-dev mailing list