[jdk11u-dev] RFR: 8218431: Improved platform checking in makefiles [v2]

[Goetz Lindenmaier]

On Tue, 31 Jan 2023 10:38:39 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> This is a quite big change, but it's all  mechanic edits.
>> It replaces the tests for platforms. The old tests
>> still work, so there is no problem in omitting some 
>> of the adaptions.
>> The new functionality added in Utels.gmk applied clean.
>> 
>> I resolved the following hunks:make/Images.gmk
>> make/RunTests.gmk
>> Simple resolve
>> 
>> make/TestImage.gmk
>> Skipped, edited code not in 11.
>> 
>> make/common/NativeCompilation.gmk
>> Two hunks did not apply.
>> Skipped first one, code not in 11.
>> Second was trivial.
>> 
>> make/copy/Copy-java.base.gmk
>> First trivial.
>> Second skipped. Two conditions checked magically :)
>> Third skipped.
>> 
>> make/hotspot/gensrc/GensrcAdlc.gmk
>> Trivial.
>> 
>> make/hotspot/ide/CreateVSProject.gmk
>> Skipped, code not in 11.
>> 
>> make/hotspot/lib/CompileDtraceLibraries.gmk
>> Trivial
>> 
>> make/hotspot/lib/CompileJvm.gmk
>> Trivial.
>> 
>> make/launcher/LauncherCommon.gmk
>> Omitted. Code not in 11.
>> 
>> make/lib/Awt2dLibraries.gmk
>> Parts applied, parts of the patched code not in 11.
>> Second and third hung to resolve were trivial.
>> 
>> make/lib/CoreLibraries.gmk
>> Trival.
>> 
>> make/lib/Lib-java.base.gmk
>> Code not in 11.
>> 
>> make/lib/Lib-jdk.hotspot.agent.gmk
>> Trivial
>> 
>> make/test/JtregNativeJdk.gmk
>> Trivial
>
> Goetz Lindenmaier 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 goetz_backport_8218431
>  - Backport 208c58c8621247023e39c98a7b81638c62daedec
>  - Move differences between current 11 and the base of the orig change to Utils.gmk
>  - Backport 16271669822c7500b60fc4da6268f682d0ba581c

Thanks for catching this! I hadn't saved this ... the editor was still open.  Fixed.
I also merged the main branch, so now the jtreg test errors should be gone.

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

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