-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
V8 nightly tests failing, svn no longer supported #9222
Comments
@ofrobots you aware of anything that went in yesterday that might be related ? |
The builds seem to be failing at the
|
I'm traveling to a conference next week so I won't be able to investigate in detail but have asked @jbajwa to take a look. |
google build tool gclient doesn't support SVN anymore, hence we are seeing this error. |
I'm working on this issue. Will update the tools/make-v8.sh script to use the git alternate. |
I updated the script to configure gclient with nodejs/v8.git repository, and performed "gclient sync". As per the documentation here, gclient sync should only pull dependencies mentioned in the DEPS file, but gclient sync seems to be updating the src as well, which is not the intended behaviour. I'm not quite sure how svn was setup before. |
Hi @s3ththompson, |
@jbajwa any progress on this ? |
@mhdawson |
@jbajwa Sorry for not hearing back on the issue. It's fall vacation time (and a holiday today). I'll ping a few people in the office tomorrow. |
landed #9393 and updated build job. Looks all good now, closing. |
The v8 test jobs started failing yesterday (the previous nightly run was ok).
See this as an example: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=ppcle-ubuntu1404,v8test=v8test/370/console
On first glance it looks like the url is pointing to the node.js repo instead of the v8 repo
And a specific example:
The text was updated successfully, but these errors were encountered: