search

Hexagon / node-telldus

Node bindings for telldus-core
Other
34 stars 10 forks source link

Does not work with Node.js 0.12 #65

Closed tonila closed 8 years ago

tonila commented 9 years ago

npm install prints following errors:

http://pastebin.com/dgvH4ud6

freber commented 9 years ago

Please fix :)

emiloberg commented 9 years ago

+1

kenderson commented 8 years ago

+1

Zebusch commented 8 years ago

I have forked this repo and made some changes. Not complete though but at least i can use it with sensors. https://github.com/Zebusch/node-telldus-zb

Perhaps someone would like to finish the work and then make a pull request :)

Hexagon commented 8 years ago

Sorry about the delay! Now there's a new branch called experimental, please clone it, test it, fork it and fix it!

@Zebusch , I decided to start over from scratch as your code didn't compile on 5.1, and because I had to learn the new changes of V8 myself. But i did get a lot of help looking at your code, and i reused some of it. Thanks :)

Hexagon commented 8 years ago

v4.2 and 5.1 seem to work fine. Trying 0.12 as soon as it has compiled.

@emiloberg , could you try it against node-red-contrib-tellstick ?

Hexagon commented 8 years ago

Well, now it's released :)

emiloberg commented 8 years ago

Well done! :star: I'll let you know if me or any of my users run in to any problems!