RFR: 8282944: GHA: Add Alpine Linux x86_64 pre-integration check [v11]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Aug 19 13:23:50 UTC 2024
On Mon, 19 Aug 2024 09:44:04 GMT, George Adams <gdams at openjdk.org> wrote:
>> Adds Alpine build CI job to the GitHub actions matrix. Currently this only builds hotspot. I can add tests too if people think that would be useful but for now I've left it as just build.
>
> George Adams has updated the pull request incrementally with two additional commits since the last revision:
>
> - improve conditional logic
> - try this
.github/workflows/main.yml line 84:
> 82:
> 83: # List of platforms to exclude by default
> 84: EXCLUDED_PLATFORMS=("alpine-linux-x64")
I'm not to fond of declaring a list like this in the middle of a code body. Perhaps you can set it up as a non-required workflow input with alpine as the default value, declared next to the default list of platforms?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20577#discussion_r1721781401
More information about the build-dev
mailing list