diasurgical / devilution

Diablo devolved - magic behind the 1996 computer game
Other
8.77k stars 919 forks source link

Add AppVeyor support #59

Closed Saibamen closed 6 years ago

Saibamen commented 6 years ago

When we will have Visual Studio 2013, 2015 or 2017 solution files we can add AppVeyor CI support to build this project on Windows.

StephenCWills commented 6 years ago

I was able to get this working with the following changes to AppVeyor's default project configuration. This doesn't disable DEP, but it still produces a successful build so should be good enough for CI.

~Environment > Build worker image~ ~Visual Studio 2017~

Build > Configuration Release

Build > Visual Studio solution or project file Diablo.sln

Build > Before build script CMD "%VS140COMNTOOLS%\..\IDE\devenv.exe" /Upgrade Diablo.dsw & exit 0

EDIT: You don't need to change the default build worker image, and doing so is, in fact, inconsistent with the call to "%VS140COMNTOOLS%\..\IDE\devenv.exe". There is no %VS150COMNTOOLS% in AppVeyor's environment so it's probably better to stick with VS2015 for devilution. If a derivative project is using VS2017, I would recommend doing a one-time upgrade and then commit the sln and vcxproj files in the repository for that project.