On Sat, 3 Feb 2024 16:22:14 GMT, George Adams <gd...@openjdk.org> wrote:
>> Now that macOS M1 executors are [available in GitHub >> actions](https://github.blog/changelog/2024-01-30-github-actions-introducing-the-new-m1-macos-runner-available-to-open-source/) >> it makes sense to move the build to run on M1 (rather than cross-compiled) >> and also enable testing on the platform. > > George Adams has updated the pull request incrementally with one additional > commit since the last revision: > > update copyright headers .github/actions/get-jtreg/action.yml line 66: > 64: fi > 65: # Build JTReg and move files to the proper locations > 66: bash make/build.sh --jdk "$JDK" We did this `JAVA_HOME_17_X64` change for https://github.com/openjdk/jdk/commit/959a61fdd483c9523764b9ba0972f59ca06db0ee, which is partially reverted during upgrade of GH actions. (...and I forgot to revert this hunk, apparently.) I wonder if it would be simpler if we switch back to boot JDK for building jtreg again, which would eliminate this hunk? Might make backports easier too. I have no strong opinion here, though. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17694#discussion_r1477130456