[jdk11u-dev] RFR: 8343786: [11u] GHA: Bump macOS and Xcode versions to macos-13 and XCode 14.3.1
Antonio Vieiro
duke at openjdk.org
Thu Nov 14 19:13:22 UTC 2024
On Thu, 14 Nov 2024 18:55:09 GMT, Antonio Vieiro <duke at openjdk.org> wrote:
> With [JDK-8340671](https://bugs.openjdk.org/browse/JDK-8340671) the GHA runner was bumped from `macos-11` to `macos-12` with minimum impact.
>
> But the `macos-12` runner image [will be fully retired by December 3rd, 2024](https://github.blog/changelog/2024-08-19-notice-of-upcoming-deprecations-and-breaking-changes-in-github-actions-runners/), so it's now convenient to bump to `macos-13`.
>
> Previous attempts to upgrade to GHA runners to `macos-13` required a long list of backports that proved difficult to review and risky to backport. These were related to the deprecation of `sprintf` (that is replaced with `snprintf` in more modern OpenJDK versions), and to passing arguments to functions without a prototype in AWT native libraries.
>
> Instead of risking to do these backports an alternate solution may be to add specific compilation flags that transform these errors to warnings. This is done by adding `-Wno-deprecated-declarations` (to bypass `sprintf` deprecation) and `-Wno-deprecated-non-prototype` to bypass passing arguments to a function without a prototype to specific parts of the makefiles for the `macosx` platform only.
>
> For easier review the PR is separated in three commits in two pushes.
>
> - First push (to verify a proper build on `macos-12` with new compilation flags)
> - The first commit adds `-Wno-deprecated-declaration` to some makefiles (`macosx` only).
> - And the second one adds `-Wno-deprecated-non-prototype` to the affected AWT libraries (again `macosx` only).
> - A second push (to ensure a proper build on `macos-13`)
> - A third commit that bumps GHA runners to `macos-13` and `xcode 14.3.1`
>
> Tier-1 tests seem to be failing randomly on `macos-13` with either a timeout or an ArrayIndexOutOfBoundsException t in `serviceability/sa/ClhsdbFindPC.java`. This is currently under investigation and may be related to [JDK-8260389](https://bugs.openjdk.org/browse/JDK-8260389) or [JDK-8277079](https://bugs.openjdk.org/browse/JDK-8277079).
>
> Solving this `tier-1` test problem may require a second PR.
Mmm... We'll need a compilation flag to build in both `macos-12` _and_ `macos-13`, as these new compilation flags are just on `XCode 14.3.1` and not on previous versions. After all we wouldn't like old `macos` pipelines failing.
I'll close the PR and add these flags.
-------------
PR Comment: https://git.openjdk.org/jdk11u-dev/pull/2965#issuecomment-2477211578
More information about the jdk-updates-dev
mailing list