RFR: Cleaner GHA build of hermetic-java-runtime branch [v2]

Jiangli Zhou jiangli at openjdk.org
Wed Apr 30 15:06:19 UTC 2025


On Wed, 30 Apr 2025 09:35:35 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Please review this simple PR which fixes build issues seen on GHA. Before this PR we see build failures in various Linux cross-builds and windows builds. This PR should fix those. Note: There are still GHA test failures, but at least we can build the branch on the main 3 platforms.
>> 
>> Thoughts?
>
> Severin Gehwolf 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 three additional commits since the last revision:
> 
>  - Revert globalDefinitions.hpp change
>  - Merge branch 'hermetic-java-runtime' into fix_build_issues_hermetic
>  - Cleaner GHA build of hermetic-java-runtime branch

src/hotspot/os_cpu/linux_aarch64/threadLS_linux_aarch64.S line 1:

> 1: // Copyright (c) 2015, 2022, Red Hat Inc. All rights reserved.

The src/hotspot/os_cpu/linux_aarch64/threadLS_linux_aarch64.S also needs to be reverted. It causes the linux-cross-build aarch64 build failures:


...
/usr/lib/gcc-cross/aarch64-linux-gnu/10/../../../../aarch64-linux-gnu/bin/ld: /home/runner/work/leyden/leyden/src/hotspot/os_cpu/linux_aarch64/threadLS_linux_aarch64.S:38: undefined reference to `Thread::_thr_current'
collect2: error: ld returned 1 exit status
* For target hotspot_variant-server_libjvm_objs_BUILD_LIBJVM_run_ld:
/usr/lib/gcc-cross/aarch64-linux-gnu/10/../../../../aarch64-linux-gnu/bin/ld: /home/runner/work/leyden/leyden/build/linux-aarch64/hotspot/variant-server/libjvm/objs/threadLS_linux_aarch64.o: in function `JavaThread::aarch64_get_thread_helper()':
/home/runner/work/leyden/leyden/src/hotspot/os_cpu/linux_aarch64/threadLS_linux_aarch64.S:34: undefined reference to `Thread::_thr_current'
...

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

PR Review Comment: https://git.openjdk.org/leyden/pull/63#discussion_r2068869527


More information about the leyden-dev mailing list