Skip to content
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

Disable XP deprecation warning when compiling with msvc v141_xp toolset #1438

Closed
withmorten opened this issue Apr 29, 2020 · 3 comments
Closed
Labels

Comments

@withmorten
Copy link
Contributor

Hi there - I often compile projects with the v141_xp toolset in VS2019 - but doing so emits the following warning:

MSB8051: Support for targeting Windows XP is deprecated and will not be present in future releases of Visual Studio.

This can be disabled by adding <XPDeprecationWarning>false</XPDeprecationWarning> within a PropertyGroup of the Project, but I haven't found a way to do so with a premake.lua script.

So then, I just now decided to just fix it rather hackily in the premake source itself, by simply adding this line to any vc project ... (my commit in my fork here):

withmorten@aedeb32

Now that I know it's possible, is there any better way this could be solved? I personally doubt anybody choosing this specific toolset wants to see the warning, but maybe it should be only enabled with a specific setting, but I honestly don't know how one would implement it in premake.

Any ideas?

Thanks for the great tool,
Morten

@withmorten withmorten added the bug label Apr 29, 2020
@starkos
Copy link
Member

starkos commented Apr 29, 2020

I would be good with always adding it by default. Could you submit a PR to add it?

@withmorten
Copy link
Contributor Author

Of course!

@starkos
Copy link
Member

starkos commented May 1, 2020

Closed by #1439.

@starkos starkos closed this as completed May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants