Skip to content

doc: update kernel and glibc reqs for PPCle - #28162

Closed
mhdawson wants to merge 3 commits into
nodejs:masterfrom
mhdawson:ppcversions
Closed

doc: update kernel and glibc reqs for PPCle#28162
mhdawson wants to merge 3 commits into
nodejs:masterfrom
mhdawson:ppcversions

Conversation

@mhdawson

Copy link
Copy Markdown
Member

We've moved master to build on centos7 for PPCle, adjust
the glibc and kernel levels accordingly.

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

We've moved master to build on centos7 for PPCle, adjust
the glibc and kernel levels accordingly.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sadly, an error occurred when I tried to trigger a build. :(

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. labels Jun 10, 2019
Comment threadBUILDING.md Outdated
Comment threadBUILDING.md
@mhdawson

Copy link
Copy Markdown
MemberAuthor

@richardlau thanks for catching that, fixed.

@mhdawson

Copy link
Copy Markdown
MemberAuthor

@sam-github can you review as well?

@mhdawsonmhdawson added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 13, 2019
@mhdawson

mhdawson commented Jun 13, 2019

Copy link
Copy Markdown
MemberAuthor

@TrottTrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rubber-stamp-ish LGTM

@sam-github

Copy link
Copy Markdown
Contributor

Even after adjusting the format of #28162 (comment), git node land says No CI runs detected, not sure why.

pullBot pushed a commit to Rachelmorrell/node that referenced this pull request Jun 13, 2019
We've moved master to build on centos7 for PPCle, adjust
the glibc and kernel levels accordingly.
PR-URL: nodejs#28162
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@richardlau

Copy link
Copy Markdown
Member

Landed in 8cdc1bd

@richardlau

Copy link
Copy Markdown
Member

Even after adjusting the format of #28162 (comment), git node land says No CI runs detected, not sure why.

nodejs/node-core-utils#354 will fix this.

BridgeAR pushed a commit that referenced this pull request Jun 17, 2019
We've moved master to build on centos7 for PPCle, adjust
the glibc and kernel levels accordingly.
PR-URL: #28162
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@BridgeARBridgeAR mentioned this pull request Jun 17, 2019
targos pushed a commit that referenced this pull request Jun 18, 2019
We've moved master to build on centos7 for PPCle, adjust
the glibc and kernel levels accordingly.
PR-URL: #28162
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Beth Griggs <Bethany.Griggs@uk.ibm.com>
@mhdawson
mhdawson deleted the ppcversions branch September 30, 2019 13:16
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.buildIssues and PRs related to build files or the CI.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@mhdawson@nodejs-github-bot@sam-github@richardlau@Trott@BethGriggs