Commit53fd0f0

committed

Honor MSVC WindowsSDKVersion if set

Add a line to the project file setting the target SDK. Otherwise, in forexample VS2017, if the default but optional 8.1 SDK is not installed thebuild will fail.Patch from Peifeng Qiu, slightly edited by me.Discussion:https://postgr.es/m/CABmtVJhw1boP_bd4=b3Qv5YnqEdL696NtHFi2ruiyQ6mFHkeQQ@mail.gmail.comBackpatch to all live branches.

1 parent0e259d4 commit53fd0f0Copy full SHA for 53fd0f0

File tree

+13

-0

lines changed

+13

-0

lines changed

Lines changed: 13 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,19 @@ EOF`
`38`	`38`	`</ItemGroup>`
`39`	`39`	`<PropertyGroup Label="Globals">`
`40`	`40`	`<ProjectGuid>$self->{guid}</ProjectGuid>`
	`41`	`+EOF`
	`42`	`+# Check whether WindowsSDKVersion env variable is present.`
	`43`	`+# Add WindowsTargetPlatformVersion node if so.`
	`44`	`+my$sdkVersion =$ENV{'WindowsSDKVersion'};`
	`45`	`+if (defined($sdkVersion))`
	`46`	`+{`
	`47`	`+# remove trailing backslash if necessary.`
	`48`	`+$sdkVersion, =~s/\\$//;`
	`49`	`+print$f<<EOF`
	`50`	`+ <WindowsTargetPlatformVersion>$sdkVersion</WindowsTargetPlatformVersion>`
	`51`	`+EOF`
	`52`	`+}`
	`53`	`+print$f<<EOF;`
`41`	`54`	`</PropertyGroup>`
`42`	`55`	`<Import Project="\$(VCTargetsPath)\\Microsoft.Cpp.Default.props" />`
`43`	`56`	`EOF`

Comments

(0)