Skip to content

[v14.x backport] build: set DESTCPU correctly for 'make binary' on Apple Silicon - #41855

Closed
evanlucas wants to merge 1 commit into
nodejs:v14.x-stagingfrom
evanlucas:backport-40147-to-v14.x
Closed

[v14.x backport] build: set DESTCPU correctly for 'make binary' on Apple Silicon#41855
evanlucas wants to merge 1 commit into
nodejs:v14.x-stagingfrom
evanlucas:backport-40147-to-v14.x

Conversation

@evanlucas

@evanlucasevanlucas commented Feb 5, 2022

Copy link
Copy Markdown
Contributor

PR-URL: #40147
Reviewed-By: Michaël Zasso targos@protonmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com
Reviewed-By: Luigi Pinca luigipinca@gmail.com
Reviewed-By: Richard Lau rlau@redhat.com

This fixes building on Apple Silicon on v14.x.

@nodejs-github-botnodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v14.x labels Feb 5, 2022
@evanlucas
evanlucasforce-pushed the backport-40147-to-v14.x branch from 05c2a15 to 68b4391CompareFebruary 5, 2022 03:38

@mhdawsonmhdawson 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.

LGTM

@evanlucasevanlucas added the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 25, 2022
@richardlau
richardlauforce-pushed the v14.x-staging branch 2 times, most recently from 30ba86c to 7f2825bCompareMarch 17, 2022 23:36
PR-URL: nodejs#40147
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Backport-PR-URL: nodejs#41855
@evanlucas
evanlucasforce-pushed the backport-40147-to-v14.x branch from 68b4391 to 1c9ebe3CompareApril 15, 2022 13:41
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95aduh95 mentioned this pull request Apr 28, 2022
juanarbol pushed a commit that referenced this pull request May 1, 2022
PR-URL: #40147
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Backport-PR-URL: #41855
@juanarbol

Copy link
Copy Markdown
Member

Landed in c73ac52 🎉

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

buildIssues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@evanlucas@nodejs-github-bot@juanarbol@lpinca@targos@richardlau@mhdawson@cheister