search

OpenVPN / tap-windows6

Windows TAP driver (NDIS 6)
Other
788 stars 237 forks source link

Reestablish AppVeyor CI #68

Closed rozmansi closed 5 years ago

rozmansi commented 5 years ago

This set of patches reestablishes the AppVeyor building.

AppVeyor has no EWDK, therefore the buildtap.py was updated to support Visual Studio 2015 and Windows Driver Kit 10 building.

Furthermore, the buildtap.py now checks for an error return status of invoked commands to stop on build errors.

mattock commented 5 years ago

The changes look good. Building with EWDK works. AppVeyor seems to have used the config in your PR and it is happy as well.

mattock commented 5 years ago

Could someone test building with Visual Studio?

mattock commented 5 years ago

I will merge this with a lazy-ACK: Appveyor is already building with WDK instead of EWDK. And as mentioned I tested the EWDK part for regressions.