Skip to content

[grpc] Fix ARM/ARM64 on Windows Build#6487

Merged
vicroms merged 1 commit intomicrosoft:masterfrom
dwcullop:dev/dwcullop/6485
May 17, 2019
Merged

[grpc] Fix ARM/ARM64 on Windows Build#6487
vicroms merged 1 commit intomicrosoft:masterfrom
dwcullop:dev/dwcullop/6485

Conversation

@dwcullop
Copy link
Member

This PR is for #6485 and includes the following changes that were required to get VcPkg to build gRPC for ARM/ARM64 on Windows

  1. Updated to v1.20.1
  2. Disables Codegen Build for ARM/ARM64 builds
  3. Rebased the "Fix UWP" patch so that it will apply properly
  4. Fixed the "Static Linking in Linux" patch because it specified an invalid option on Windows builds
  5. Added patch to "#undef" a macro name collision
  6. Added GDI32.lib to the list of standard libraries linked to Windows

Also: Thank you 🙏🙏🙏 so much for vcpkg! It has literally saved me an immeasurable amount of time and saved me from copious amounts of stress.

This PR includes the following changes that are required to get gRPC to build for ARM/ARM64 on Windows
1) Updated to v1.20.1
2) Disables Codegen Build for ARM/ARM64 builds
3) Rebased the "Fix UWP" patch so that it will apply properly
4) Fixed the "Static Linking in Linux" patch because it specified an invalid option on Windows builds
5) Added patch to "#undef" a macro name collision
6) Added GDI32.lib to the list of standard libraries linked to Windows
@vicroms vicroms merged commit 2fda34b into microsoft:master May 17, 2019
@vicroms
Copy link
Member

vicroms commented May 17, 2019

Thanks for the PR!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants