search

OpenZWave / node-openzwave-shared

OpenZWave addon for Node.js (all versions) including management and security functions
Other
199 stars 113 forks source link

Visual Studio updates #200

Closed jbreslin closed 6 years ago

jbreslin commented 6 years ago

Is anyone working on bringing the windows implementation up to date?

We have worked hard and resolved a number of compiler (vcbuild v msbuild) issues but still have more to resolve and do not want to waste time if this is a dead duck or others are working on it. There are a load of warnings in addition to the errors and I have no idea how to start without compromising someone else.

My objective is to get a node-openzwave contrib in a decent state for integration with node-red.

Happy to help or act in support of anyone who will lead.

ekarak commented 6 years ago

:+1:

ekarak commented 6 years ago

Can you please elaborate on 'windows implementation'? there is no platform-specific implementation, only a windows build script, all the rest should (hopefully) be platform-agnostic. BTW, version 1.4.3 was just released. This includes the fix in #222 and adds Appveyor CI support for Windows builds.

ekarak commented 6 years ago

resolving stale issue