RFR: 8318669: Target OS detection in 'test-prebuilt' makefile target is incorrect when running on MSYS2
Erik Joelsson
erikj at openjdk.org
Tue Oct 24 21:22:29 UTC 2023
On Tue, 24 Oct 2023 10:29:32 GMT, Frederic Thevenet <fthevenet at openjdk.org> wrote:
> This PR addresses OS detection not working properly when running test-prebuilt on MSYS2, which causes some Windows specific tests to misbehave or fail in Github Actions.
make/RunTestsPrebuilt.gmk line 162:
> 160: OPENJDK_TARGET_OS_ENV := windows.cygwin
> 161: else
> 162: ifeq ($(UNAME_OS), MINGW64)
Instead of nesting, this inside the else body, creating more nesting below, make supports
else ifeq ...
By using that you can keep the else body with `unix` on the same indentation level.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16343#discussion_r1370823391
More information about the build-dev
mailing list