Closed richardlau closed 1 year ago
My guess is that it's related to the removal of ninja binaries from depot_tools at the end of January 2023:
What do I need to do?
We have already added the ninja CIPD package to all known projects with DEPS. Running
gclient sync
is all you have to do in that case.
V8's DEPS in Node.js does contain a cipd entry for ninja 🤷 : https://github.com/nodejs/node/blob/86362b7877cab3d8385521a3c9dabffe4f6f0a9b/deps/v8/DEPS#L247-L256
10:32:11 - Oterweise, Add Ninja to your PATH *after* depot_tools.
One option, if we're okay using our own ninja binaries instead of Google's on x64, might be to use the https://github.com/nodejs/build/tree/main/ansible/roles/ninja role, which compiles the ninja binary that we're using for ppc64le and s390x, on x64. That would involve, at least:
when
clause, or extending it to include x64
, from https://github.com/nodejs/build/blob/fd1ced3c84f14c079c7cf4f14cb88bb4c926f7ac/ansible/roles/build-test-v8/meta/main.yml#L5-L10/home/iojs/build-tools
to the PATH. Currently on ppc64le and s390x that's done in tools/make-v8.sh
but I think that should really be done in the job (if a user runs tools/make-v8.sh
on their own system they are highly unlikely to have /home/iojs/build-tools
). Also setting it in the job means we won't have to update tools/make-v8.sh
in every supported release line.Shouldn't we just install ninja from ubuntu's repo?
Shouldn't we just install ninja from ubuntu's repo?
That's an option too. I don't think we've hit compatibility issues with ninja like we have with gn.
Shouldn't we just install ninja from ubuntu's repo?
That's an option too. I don't think we've hit compatibility issues with ninja like we have with gn.
https://github.com/nodejs/build/pull/3305 installs ninja-build
from the Linux distro's package repository.
https://ci.nodejs.org/job/node-test-commit-v8-linux/ has been failing for some time on the Intel benchmark machine, https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu1804-intel-64,v8test=v8test/buildTimeTrend
While a lot of the failures are https://github.com/nodejs/build/issues/3050, some builds, e.g. https://ci.nodejs.org/job/node-test-commit-v8-linux/5191/nodes=benchmark-ubuntu1804-intel-64,v8test=v8test/console manage to pull down deps but then error:
This is odd -- my understanding was that for x64 ninja came from Google's depot_tools/cipd.
It looks like the last successful V8 CI on the benchmark machine was 29 days ago: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu1804-intel-64,v8test=v8test/5137/
cc @nodejs/v8-update