[jdk17u-dev] RFR: 8296812: sprintf is deprecated in Xcode 14 [v2]

[Severin Gehwolf]

On Thu, 18 Jan 2024 20:22:34 GMT, Elif Aslan <duke at openjdk.org> wrote:

>> https://developer.apple.com/news/?id=y5mjxqmn
>> 
>> Starting November 1, 2023, the Apple notary service will no longer accept uploads from altool or Xcode 13 or earlier. 
>> 
>> 
>> This is not a clean backport and this pull request contains a backport of the closed bug [JDK-8296812](https://bugs.openjdk.org/browse/JDK-8296812) commit [f1522c648c2e](https://github.com/openjdk/jdk/commit/478ef389dc3767edfbe21d10a7f7f1522c648c2e).
>> 
>> Conflicts includes:
>> 
>> -  whitespace conflicts on src/hotspot/share/utilities/utf8.cpp
>> 
>> -  missing src/hotspot/share/jfr/support/jfrSymbolTable.cpp 
>> 
>> -  context difference on src/hotspot/cpu/aarch64/vm_version_aarch64.cpp
>> 
>> The sprintf is deprecated in Xcode 14 and the use of it causing building failure. The build could pass if warnings are disabled for codes that use sprintf method. For the long run, the sprintf could be replaced with snprintf. This patch is trying to check if snprintf could be used.
>> 
>> Testing:
>> 
>> GHA tested.
>> Successfull tier1,tier2,tier3 and jck tests for linux platforms.
>> Successfull tier1 and tier2 for windows.
>
> Elif Aslan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into JDK-8296812
>  - add CPU back
>  - remove CPU_STXR_PREFETCH
>  - 8296812: Backport of 478ef389dc3767edfbe21d10a7f7f1522c648c2e

There is a new PR: https://github.com/openjdk/jdk17u-dev/pull/2662

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1930#issuecomment-2209296556