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

win, build: arm64 port script and doc changes #25995

Closed
wants to merge 4 commits into from

Conversation

jkunkee
Copy link
Contributor

@jkunkee jkunkee commented Feb 7, 2019

This change lays the groundwork for ARM64 Windows ("Windows 10 on ARM") builds of Node.js.

Build script and configuration files gets new ARM64 Windows sections. The arch-specific check for NASM is scoped to the architectures that rely on it.

The bootstrap script arguments for Visual Studio tool installation are expanded to include ARM64 tools.

Documentation is updated to reflect all of these changes.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
    • ARM64 vcbuild requires other changes, but Windows x64 passes.
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform. labels Feb 7, 2019
@jkunkee jkunkee force-pushed the arm64-win-build branch 2 times, most recently from 6d4196a to c1f2ce5 Compare February 7, 2019 23:24
configure.py Show resolved Hide resolved
BUILDING.md Outdated Show resolved Hide resolved
@vielmetti
Copy link
Contributor

@jkunkee - talk to me about what you might need for arm64 CI/CD for Windows 10? (perhaps offline, perhaps here) I have some resources through @WorksOnArm but we don't yet have Windows 10 for Arm up on any servers.

@jkunkee
Copy link
Contributor Author

jkunkee commented Feb 11, 2019

@vielmetti, in general we need machines--virtual or physical--running Windows 10 on ARM and available on demand as a service for the general public. I'll ping you via email about it.

BUILDING.md Outdated Show resolved Hide resolved
@jkunkee jkunkee force-pushed the arm64-win-build branch 2 times, most recently from 3c65a55 to 333ea47 Compare February 15, 2019 01:24
@jkunkee jkunkee force-pushed the arm64-win-build branch 2 times, most recently from 1abe49f to 7083224 Compare February 20, 2019 22:29
@joaocgreis
Copy link
Member

joaocgreis commented Mar 4, 2019

joaocgreis pushed a commit that referenced this pull request Mar 4, 2019
joaocgreis pushed a commit that referenced this pull request Mar 4, 2019
joaocgreis pushed a commit that referenced this pull request Mar 4, 2019
joaocgreis pushed a commit that referenced this pull request Mar 4, 2019
@joaocgreis
Copy link
Member

Landed in d57c526...26cfaf4
Thanks @jkunkee!

@joaocgreis joaocgreis closed this Mar 4, 2019
@jkunkee jkunkee deleted the arm64-win-build branch March 8, 2019 02:39
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Mar 12, 2019
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Mar 12, 2019
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Mar 12, 2019
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Mar 12, 2019
BethGriggs pushed a commit that referenced this pull request Apr 16, 2019
BethGriggs pushed a commit that referenced this pull request Apr 16, 2019
BethGriggs pushed a commit that referenced this pull request Apr 16, 2019
BethGriggs pushed a commit that referenced this pull request Apr 16, 2019
@BethGriggs BethGriggs mentioned this pull request May 1, 2019
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. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants