Node v9+ support
alexcastillo opened this issue ยท 7 comments
Hey Uri!
It seems node-lsl does not support node v9+ because of its ffi
dependency. It looks they last published a new version 2 years ago.
I was looking into other alternatives for node bindings and found:
https://www.npmjs.com/package/fastcall
Dano mentioned N-API and WASM.
What are your thoughts?
I'm doing a bunch of work on my fork: https://github.com/makebrainwaves/node-lsl
Since fastcall is supposed to be a drop-in replacement for node-ffi I can play around with adding that and, if it works, merge those changes back to this repo.
BTW guys, I think we can make this the official liblsl Node wrapper eventually ๐ฅ
Sounds like a plan, go Dano!
Awesome!
Ended up going the n-api route for ffi and ref, have the results on a fork here: working on node v12 https://github.com/CalvinLeather/node-lsl if anyone wants to use before I clean up and possible PR into this repo.
Is this repo open for PRs? I've been using this repo on a couple projects, wouldn't mind maintaining a bit while I use it.
@CalvinLeather yes, I can add you as a collaborator if you are open to maintaining it
Sounds good, yeah I will be working with it for the next year or so at least. I'll PR in a bit. Thanks!
@CalvinLeather any plan to push your PR ? would be nice to have this update on npm.