-
Notifications
You must be signed in to change notification settings - Fork 167
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
ci,libuv: http://ci.nodejs.org/downloads/zos/gyp.tar.gz? #2172
Comments
The original upstream hasn't been maintained for some time now. I've copied https://github.com/nodejs/node/tree/master/tools/gyp to https://github.com/libuv/gyp after stripping it down a bit. Let's point users to that. I'll work on getting it tested on the CI, too. Refs: libuv/help#126 Refs: nodejs/build#2172
Similar question for the Windows buildbots:
|
I have no idea why the macOS builds would be pulling down the http://ci.nodejs.org/downloads/zos/gyp.tar.gz as that's presumably a copy of gyp with patches for z/OS.
The above last modified puts this in the timeframe when z/OS support was initially upstreamed to gyp (https://chromium-review.googlesource.com/c/external/gyp/+/644712/). As far as I'm aware none of the z/OS patches for gyp are in Node.js' |
cc @nodejs/platform-s390 |
Another reason for using the tarball on z/OS is that the z/OS port of git doesn't support https URLs: https://forum.rocketsoftware.com/t/git-and-https-repositories/482. |
Re macOS + z/OS: 🤷♂ another Jenkins mystery we'll never solve I suspect. Let's get that switched, perhaps a genuine mistake on someone's part. Re Windows, what's the problem there? Don't you need to add in a GYP to make it compile with GYP? IIRC we setup a bunch of the libuv build machines with static GYP installs that we copied into place. Re GYP version:
Hopefully we'll have one true GYP sooner than later. I want to cut a node-gyp@7 with that PR merged soon. |
That it's not cloned from anywhere but copied from the local file system. It makes me wonder how old that copy of gyp is. :-) FWIW, the consensus within the libuv team is that we'll just drop support for GYP and move the build files downstream so I'll go ahead and close out the issue. Cheers! |
build/ansible/roles/jenkins-worker/tasks/main.yml Lines 207 to 209 in 4058b56
although I have come across a GitHub mirror if we wanted to pull down via git+ssh: https://github.com/chromium/gyp |
Some of the libuv buildbots seem to use a... bespoke?... copy of gyp:
https://ci.nodejs.org/job/libuv-test-commit-osx/1748/nodes=osx1011/console
Does anyone know/remember why things are set up this way? I was thinking of extracting node's tools/gyp to https://github.com/libuv/gyp (so people can still build with a somewhat maintained version of gyp) and then switch over the CI to use that.
(Libuv's readme currently points people to chromium's obsolete copy of gyp.)
The text was updated successfully, but these errors were encountered: