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

build: handle aarch64 as arm64 #5191

Merged
merged 1 commit into from
Feb 11, 2016

Conversation

jbergstroem
Copy link
Member

Makefile is now consistent with configure on how we treat aarch64.
Refs #5175.

/r=@nodejs/build.

@jbergstroem jbergstroem added the build Issues and PRs related to build files or the CI. label Feb 11, 2016
@rvagg
Copy link
Member

rvagg commented Feb 11, 2016

lgtm

@bnoordhuis
Copy link
Member

LGTM

`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: nodejs#5191
Fixes: nodejs#5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@jbergstroem jbergstroem merged commit f167207 into nodejs:master Feb 11, 2016
@jasnell
Copy link
Member

jasnell commented Feb 11, 2016

LGTM

rvagg pushed a commit that referenced this pull request Feb 15, 2016
`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: #5191
Fixes: #5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
stefanmb pushed a commit to stefanmb/node that referenced this pull request Feb 23, 2016
`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: nodejs#5191
Fixes: nodejs#5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@MylesBorins
Copy link
Contributor

@nodejs/lts should we include this in v4.4.0?

@rvagg
Copy link
Member

rvagg commented Feb 24, 2016

risk is very low so I'd say yes to lts

MylesBorins pushed a commit that referenced this pull request Mar 1, 2016
`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: #5191
Fixes: #5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit that referenced this pull request Mar 1, 2016
`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: #5191
Fixes: #5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit that referenced this pull request Mar 2, 2016
`Makefile` is now consistent with `configure` regarding
how we treat aarch64.

PR-URL: #5191
Fixes: #5175
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants