-
Notifications
You must be signed in to change notification settings - Fork 74
Fix Loong64 #181
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
base: main
Are you sure you want to change the base?
Fix Loong64 #181
Conversation
Thank you very much for your solution. I think the error may be caused by using an outdated build environment. The cross-compilation toolchain used in the build log (https://unofficial-builds.nodejs.org/logs/202507290214-v24.4.1/loong64.log) has been replaced with the cross-compilation toolchain included with Ubuntu:24.04 in #172. Old cross-compilation toolchain(ubuntu:22.04, python-3.8)
New cross-toolchain(ubuntu:24.04 python-3.10) (https://github.com/nodejs/unofficial-builds/blob/main/recipes/loong64/run.sh#L23)
Based on the build log (https://unofficial-builds.nodejs.org/logs/202507290214-v24.4.1/loong64.log), the old build environment is still being used. |
Ahh, OK, so it's trying to build the images but failing and we're not noticing. If you look at the the github-webhook.log then you'll see the output from the last deploy at the bottom along with any errors: https://unofficial-builds.nodejs.org/logs/github-webhook.log (I've just xz compressed the current one so it's currently https://unofficial-builds.nodejs.org/logs/github-webhook.log.xz but the next one should be .log).
I think for the version of |
I guess the server is just not updated with this repository (the files in the repository are probably newer) so we get python error (#176) because older version is running.
But commands addgroup and adduser have become broken in new version anyway.
This pull request fixes it and makes the recipe work again.