Skip to content
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

tools: unify make-v8.sh for ppc64le and s390x #31628

Merged
merged 1 commit into from
Feb 7, 2020

Conversation

richardlau
Copy link
Member

Refactor tools/make-v8.sh to minimise differences between the
ppc64le and s390x paths to allow us to enable ccache on the
ppc64le machines in the CI.

This needs to land in all supported release lines tested on before
we can reenable ccache on the centos7-ppcle machines (i.e.
reland nodejs/build#1927 which was backed out because the
divergent ppc64le and s390x paths through this script
broke node-test-commit-v8-linux (nodejs/build#1940)).

Refs: nodejs/build#1927
Refs: nodejs/build#1940

cc @nodejs/platform-ppc @nodejs/platform-s390

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency. labels Feb 4, 2020
@richardlau
Copy link
Member Author

V8-CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/2844/
(benchmark-ubuntu1604-intel-64 and centos7-ppcle are expected to fail due to #30152)

@richardlau richardlau added ppc Issues and PRs related to the Power architecture. s390 Issues and PRs related to the s390 architecture. labels Feb 4, 2020
@richardlau richardlau added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. lts-watch-v10.x labels Feb 5, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 7, 2020

Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: nodejs#31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
@richardlau
Copy link
Member Author

Landed in d213f21.

@richardlau
Copy link
Member Author

This needs to land in all supported release lines tested on before
we can reenable ccache on the centos7-ppcle machines (i.e.
reland nodejs/build#1927 which was backed out because the
divergent ppc64le and s390x paths through this script
broke node-test-commit-v8-linux (nodejs/build#1940)).

@nodejs/releasers Any objections to me cherry-picking the commit across to the staging branches?

codebytere pushed a commit that referenced this pull request Feb 17, 2020
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
@codebytere codebytere mentioned this pull request Feb 17, 2020
BethGriggs pushed a commit that referenced this pull request Feb 26, 2020
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
@BethGriggs
Copy link
Member

@richardlau, the commit is now on v13.x and v12.x-staging, but it didn't land cleanly on v10.x. I'll add the backport-requested label for now.

@richardlau
Copy link
Member Author

@BethGriggs d213f21 lands cleanly on v10.x-staging if cf5ce2c is cherry-picked first.

BethGriggs pushed a commit that referenced this pull request Mar 2, 2020
Refactor `tools/make-v8.sh` to minimise differences between the
`ppc64le` and `s390x` paths to allow us to enable `ccache` on the
`ppc64le` machines in the CI.

PR-URL: #31628
Refs: nodejs/build#1927
Refs: nodejs/build#1940
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
@BethGriggs BethGriggs mentioned this pull request Mar 12, 2020
@codebytere codebytere mentioned this pull request Mar 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. ppc Issues and PRs related to the Power architecture. s390 Issues and PRs related to the s390 architecture. tools Issues and PRs related to the tools directory. v8 engine Issues and PRs related to the V8 dependency.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants