-
-
Notifications
You must be signed in to change notification settings - Fork 617
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
Compiler Version support for Visual Studion 2017+ #1655
Conversation
modules/vstudio/vs2010_vcxproj.lua
Outdated
elseif cfg.compileas == "Module" then | ||
elseif cfg.compileas == "Module" then | ||
m.element("CompileAs", condition, "CompileAsCppModule") | ||
elseif cfg.compileas == "ModulePartition" then | ||
elseif cfg.compileas == "ModulePartition" then | ||
m.element("CompileAs", condition, "CompileAsCppModuleInternalPartition") | ||
elseif cfg.compileas == "HeaderUnit" then | ||
elseif cfg.compileas == "HeaderUnit" then |
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.
Looks like this got caught in light ending fixes.
src/_premake_init.lua
Outdated
"Module", | ||
"ModulePartition", | ||
"HeaderUnit" | ||
"Module", | ||
"ModulePartition", | ||
"HeaderUnit" |
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.
Same as above. This was caught in line ending fixes.
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.
Nice, thanks!
Hey @nickclark2016, you forgot to add Btw. the latest version of docusaurus introduced some kind of automation for sidebars. It's not perfect yet but I'll try this feature in time. |
@KyrietS Ahh crap, yup, I'll do that. |
@nickclark2016 *cough* *cough* 🙃 |
Oh crap, totally slipped my mind. Standby |
Updated sidebar to include toolsversion link
What does this PR do?
Adds compiler version support for VS 2017 and newer
How does this PR change Premake's behavior?
No breaking changes, all new functionality.
Syntax
This will add the following to the globals group in VS 2017 and newer project files.
It is worth noting that this does no checks for legal tool versions. That is on the user. I believe it would be more hassle than it is worth to go check for all installed tool version, then validate user input against that.
Anything else we should know?
Closes #1612
Did you check all the boxes?
closes #XXXX
in comment to auto-close issue when PR is merged)You can now support Premake on our OpenCollective. Your contributions help us spend more time responding to requests like these!