-
-
Notifications
You must be signed in to change notification settings - Fork 314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TKG build failure on temurin-build
Github actions checks
#4914
Comments
Noting: https://github.com/adoptium/temurin-build/blob/master/.github/workflows/build.yml#L95-L108 it is trying to run the smoke test as part of PR testing |
@llxia - FYI, this is related to recent improvements to how we handle dependencies we download. Looks like json.jar file is in an unexpected location now for run-aqa (github workflows). |
https://github.com/adoptium/temurin-build/actions/runs/7128536879/job/19455522686?pr=3538#step:8:407 We need to set |
LIB_DIR has the default value, which should not need to be explicitly export. The change https://github.com/adoptium/TKG/pull/476/files in settings.mk is invisible to make compile process. |
I have rerun the github runner tests seems initial issue is fixed. https://github.com/adoptium/temurin-build/actions/runs/7172315110/job/19572418555#step:8:411. However similar issue happened when compile the tests, https://github.com/adoptium/temurin-build/actions/runs/7172315110/job/19572418555#step:8:552. In the locally run both compile process with build_tool.xml and build_test.xml are run correctly with -DLIB_DIR=/Users/.../test/aqa-tests/TKG/../TKG/lib. Reopen untill all issues be addressed. A little bit confused as in run-aqa 'make compile' is one process. |
Temurin buil PR checks are currnetly failiung when building TKG material:
Copying this from my slack message:
Something in that area is causing a failure on the temurin-build automated PR checks and it wasn't immediately obvious from looking at recent PRs what it might be: adoptium/temurin-build#3538 (comment)
Relevant run of the check: https://github.com/adoptium/temurin-build/actions/runs/7128536879/job/19455522686?pr=3538
The text was updated successfully, but these errors were encountered: