search

mathiask88 / node-snap7

node.js wrapper for snap7
MIT License
163 stars 59 forks source link

Build error on nodejs 10 and windows #40

Closed Apollon77 closed 6 years ago

Apollon77 commented 6 years ago

see: https://ci.appveyor.com/project/Apollon77/iobroker-s7/build/job/ajdglh0wjyxicds8#L35

mathiask88 commented 6 years ago

Yes, I know. Will be patched next week :)

mathiask88 commented 6 years ago

https://github.com/mathiask88/node-snap7/tree/v1.0.1

Apollon77 commented 6 years ago

Yes node 10 now works ... but nodejs 6 no longer on linux/macos!! --> https://github.com/mathiask88/node-snap7/issues/42

mathiask88 commented 6 years ago

This is odd. Passing on my CI at least https://travis-ci.org/mathiask88/node-snap7 :S

Apollon77 commented 6 years ago

Strange ... BTW it only affects linux&mac ... windows works. Do you have any idea?

Apollon77 commented 6 years ago

Ok, seems solved ... I had a "--build-from-source" in the install call and then I got that error, but now without that and using normal "npm install" that uses prebuild binaries from you everything works. still strange, but solved for now