Skip to content

Commit

Permalink
Fix AppVeyor for VS 2017
Browse files Browse the repository at this point in the history
The idiots at Microsoft, in their infinte wisdom, have suddenly decided that if it looks like you have a CMake configuration in your source tree that you want to use CMake instead of MSBuild and automatically start up a background CMake and makefile run.

This, of course, means that it fails because CMake can not find the external dependencies.

It also means AppVeyor hangs until we timeout because, as a background process, AppVeyor never sees the job is running. To make matters worse, this seems to actually try to compile the full source, at a glacially slow rate.

And, with all due praises to their godly infalibility, Microsoft has not provided any means to stop using CMake.

So, while it's not a perfect upgrade, so there may be minor differences from what you'll see using the full IDE interactively, we can override the PlatformToolset to use v141 while using MSBuild with the Visual Studio 2013 project files.

All Hail Microsoft!
  • Loading branch information
GregoryLundberg committed Feb 22, 2018
1 parent 2180644 commit 70d1d9f
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions .appveyor.vs2017.yml
Expand Up @@ -15,14 +15,9 @@ before_build:
- 7z x VC15.ZIP
- ren external-VC15 external
- cd wesnoth
- mkdir projectfiles\VC15
- xcopy /E /I /H /Y projectfiles\VC12 projectfiles\VC15
- cd projectfiles\VC15
- "\"c:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\Community\\Common7\\IDE\\devenv.exe\" wesnoth.sln /upgrade"
- cd ..\..
- set PATH=C:\projects\external\dll;%PATH%
build_script:
- msbuild projectfiles\VC15\wesnoth.sln /logger:"C:\Program Files\AppVeyor\BuildAgent\Appveyor.MSBuildLogger.dll"
- msbuild projectfiles\VC12\wesnoth.sln /property:PlatformToolset=v141 /m /logger:"C:\Program Files\AppVeyor\BuildAgent\Appveyor.MSBuildLogger.dll"

artifacts:
- path: wesnoth.exe
Expand Down

0 comments on commit 70d1d9f

Please sign in to comment.