search

trailofbits / cb-multios

DARPA Challenges Sets for Linux, Windows, and macOS
https://blog.trailofbits.com/2016/08/01/your-tool-works-better-than-mine-prove-it/
MIT License
520 stars 103 forks source link

Use ninja on Windows CI #65

Closed woodruffw closed 5 years ago

woodruffw commented 5 years ago

64 makes macOS use ninja; Windows should do the same on AppVeyor.

ekilmer commented 5 years ago

I believe Windows already uses Ninja for at least the clang build here https://github.com/trailofbits/cb-multios/blob/561f2369e20e2727376028578631a7363ef9d2be/.appveyor.yml#L26-L30

ekilmer commented 5 years ago

Admittedly, it only uses Ninja for the clang build https://github.com/trailofbits/cb-multios/blob/561f2369e20e2727376028578631a7363ef9d2be/build.ps1#L39-L43

However, it uses the default CMake generator for the (broken) MSVC compiler build

woodruffw commented 5 years ago

Yep, my bad. I was looking at the AppVeyor scrollback and didn't see "Ninja", so I assumed it was using the default backend.