[jdk11u-dev] RFR: 8344458: [11u] Add initial support for building with XCode 14 [v3]

Antonio Vieiro duke at openjdk.org
Tue Nov 19 14:08:39 UTC 2024


> An implementation of [JDK-8344458](https://bugs.openjdk.org/browse/JDK-8344458) that adds conditional support for building on `macos` with `XCode 14` while keeping compatibility with previous `XCode` versions.
> 
> The PR is separated in three commits for easier review:
> 
> - First commit adds a new `--enable-xcode14` configuration flag (which is currently disabled in GHA) and two additional variables:
>     - `CFLAGS_XCODE14_DEPR_DECLARATIONS` (empty on current XCode versions, set to and `-Wno-deprecated-declarations` when `--enable-xcode14`is used)
>     - And `CFLAGS_XCODE14_DEPR_NON_PROTOTYPE` (empty on current XCode versions, set to `-Wno-deprecated-non-prototype` when `--enable-xcode14` is used).
> - The second commit applies `CFLAGS_XCODE14_DEPR_DECLARATIONS` to those parts of the codebase that use the deprecated `sprintf` function (i.e., avoiding `sprintf` usage errors in XCode 14).
> - The third commit applies `CFLAGS_XCODE14_DEPR_NON_PROTOTYPE` to those parts of AWT lib that generate a compilation error (i.e., avoiding the `passing arguments to a function without prototype`).
> 
> Since the new flag `--enable-xcode14` is not set **the build should run exactly the same on the current XCode versions** and, consequently, the GitHub checks should pass on all platforms, including the current `macos-12` & `XCode 13.4.1`.

Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision:

  Removed doc instructions as per review

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

Changes:
  - all: https://git.openjdk.org/jdk11u-dev/pull/2966/files
  - new: https://git.openjdk.org/jdk11u-dev/pull/2966/files/fec6eed5..0a5ecb00

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk11u-dev&pr=2966&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk11u-dev&pr=2966&range=01-02

  Stats: 3 lines in 2 files changed: 0 ins; 3 del; 0 mod
  Patch: https://git.openjdk.org/jdk11u-dev/pull/2966.diff
  Fetch: git fetch https://git.openjdk.org/jdk11u-dev.git pull/2966/head:pull/2966

PR: https://git.openjdk.org/jdk11u-dev/pull/2966


More information about the jdk-updates-dev mailing list