quixdb / squash

Compression abstraction library and utilities
https://quixdb.github.io/squash/
MIT License
407 stars 53 forks source link

Add Visual Studio 2017 to AppVeyor #233

Closed jibsen closed 7 years ago

jibsen commented 7 years ago

It looks like squash builds with VS 2017 on AppVeyor now, the only change needed was to select a specific build image with the new compiler on. A few things to decide before adding this:

For now I commented out VS 2013, do we wish to continue having it in the build matrix? On one hand it's getting old, and fewer configurations means faster CI runs, on the other hand some things might break on older compilers moving forward. What do you think?

Also, as mentioned in #185, Clang/C2 appears to be dead, so perhaps we should remove the commented out section for it?

nemequ commented 7 years ago

Looks like the latest build failed on clang.

Once that's resolved, feel free to push it to master (or I can do it if you prefer).

nemequ commented 7 years ago

Realized that's for a different branch; pushed this one to master. Thanks :)

94043 commented 6 years ago