-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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] Add MSBuild option to setup.py #6724
Conversation
✅ Deploy Preview for docsite-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
I did notice some unrelated code adjustments (texture, device, etc). Was it mistakenly merged with some other PRs?
ah yes... accidentally merged in the headers cleanup |
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.
Let's make sure CXX standard remain unchanged in this PR, otherwise LGTM!
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.
Thanks!
Turns out it works flawlessly with scikit-build. We used to only provide Ninja because we roll the build commands ourselves.
Set environment variable TAICHI_USE_MSBUILD to 1 or ON to use MSBuild + MSVC. This will also provide a seamless user experience in Visual Studio.