rwaldron / galileo-io

Intel Galileo & Intel Edison IO Plugin for Johnny-Five
http://johnny-five.io
MIT License
101 stars 26 forks source link

fs: missing callback Error #3

Closed luigif closed 10 years ago

luigif commented 10 years ago

Using the library with a recent version of node.js returns many fs errors, like:

fs: missing callback Error: ENOENT, open '/sys/class/gpio/gpio40/drive'

This is due to fs module functions now requiring a callback (https://github.com/joyent/node/issues/5005)

rwaldron commented 10 years ago

Unfortunately, I'm away from home (where my galileo is) until later this week, but a patch is certainly welcome and I will review and test as soon as possible.

Note to self: https://communities.intel.com/message/221298#thread-message-221374

georgemck commented 10 years ago

Looks like another one: fs: missing callback Error: EINVAL, write I was using digitalWrite() when it came out. Node v0.10.25 ; NPM v1.3.24

maheshbalan commented 10 years ago

Any updates as to when a patch will be available ?. Itching to test some Galileo use cases with node.js..

rwaldron commented 10 years ago

Whoops, completely forgot about this, thanks for the reminder. I'll push a patch tomorrow AM

maheshbalan commented 10 years ago

Many thanks! On Apr 30, 2014, at 6:25 PM, Rick Waldron notifications@github.com wrote:

Whoops, completely forgot about this, thanks for the reminder. I'll push a patch tomorrow AM

— Reply to this email directly or view it on GitHub.

rwaldron commented 10 years ago

no-op'ing the callbacks should be sufficient. Let me know if there are any issues.

maheshbalan commented 10 years ago

Galileo is in the office. Will try it first thing tomorrow! On Apr 30, 2014 7:20 PM, "Rick Waldron" notifications@github.com wrote:

no-op'ing the callbacks should be sufficient. Let me know if there are any issues.

— Reply to this email directly or view it on GitHubhttps://github.com/rwaldron/galileo-io/issues/3#issuecomment-41873642 .

maheshbalan commented 10 years ago

Fixed! Should have read the article by Sergey Kiselev on the GPIO to pin mappings. Wasted an hour trying it via filesystem shell commands before I realized the pin # to sysfs GPIO # - Anyway, tried galileo-io and got the blinking LED going right away.

rwaldron commented 10 years ago

Anyway, tried galileo-io and got the blinking LED going right away.

Awesome!

I'm certain there will be bugs—I haven't had as much time as I'd like to put it through the paces, so please file them when you find them :)

maheshbalan commented 10 years ago

Will do! On May 2, 2014 7:41 AM, "Rick Waldron" notifications@github.com wrote:

Anyway, tried galileo-io and got the blinking LED going right away.

Awesome!

I'm certain there will be bugs—I haven't had as much time as I'd like to put it through the paces, so please file them when you find them :)

— Reply to this email directly or view it on GitHubhttps://github.com/rwaldron/galileo-io/issues/3#issuecomment-42038835 .