-
Notifications
You must be signed in to change notification settings - Fork 55
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
Geode-6054: Build with VS2017 and new gtest #411
Geode-6054: Build with VS2017 and new gtest #411
Conversation
Co-authored-by: Ernest Burghardt <eburghardt@pivotal.io>
Co-authored-by: Ernest Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io>
- No longer need the D_SILENCE_TR1_NAMESPACE_DEPRECATION_WARNING workaround for VS 2017
All requested changes completed. |
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.
Without a packer script to build a windows image I’m not keen on merging this.
Agreed. Should I close this PR for now? |
No. I would just get a PR open for the image so others can test this change. As is there’s is no guidance on how to build a viable VS 2017 environment. After that PR is accepted then you could merge this one. |
No description provided.