-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
[cmake] improved stdalign.h workaround for windows build #32777
Conversation
Adhoc windows/grpc_portability run http://sponge/68f4cc3e-9e17-4cd4-9b28-48bc97e3eb87 |
@rem Always use the newest Windows 10 SDK available. | ||
@rem A new-enough Windows 10 SDK that supports C++11's stdalign.h is required | ||
@rem for a successful build. | ||
@rem For some reason, by default cmake together with visual studio generator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: Can probably rephrase the "For some reason" bit now, since we know why it's happening. (It prefers the Windows 10 SDK whose version number happens to match the host OS's version number.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
dc44d88
to
70c622c
Compare
BTW, won't help us until it's released and such, but future CMake should hopefully be better here. |
Bump to kick the title check. |
70c622c
to
d84899e
Compare
Improved the workaround from grpc#32764 as suggested in grpc#32764 (comment). Once merged, I'll backport to 1.54.x and 1.53.x together with grpc#32764 (these have switched to VS2019)
Improved the workaround from grpc#32764 as suggested in grpc#32764 (comment). Once merged, I'll backport to 1.54.x and 1.53.x together with grpc#32764 (these have switched to VS2019)
Improved the workaround from grpc#32764 as suggested in grpc#32764 (comment). Once merged, I'll backport to 1.54.x and 1.53.x together with grpc#32764 (these have switched to VS2019)
Improved the workaround from #32764 as suggested in #32764 (comment). Once merged, I'll backport to 1.54.x and 1.53.x together with #32764 (these have switched to VS2019)
Improved the workaround from #32764 as suggested in #32764 (comment).
Once merged, I'll backport to 1.54.x and 1.53.x together with #32764 (these have switched to VS2019)