jduncanator / node-diskusage

Node.js bindings around platform specific disk usage interfaces.
MIT License
155 stars 51 forks source link

diskusage does not work with Node JS 4.x #7

Closed jschrab closed 9 years ago

jschrab commented 9 years ago

It would appear this is because of the https://github.com/node-ffi/node-ffi dependency: https://github.com/node-ffi/node-ffi/issues/245

I mention this here in hopes that the node-diskusage repo is updated to use the next release of node-ffi, once it's ready.

dimd13 commented 9 years ago

+

jduncanator commented 9 years ago

Thanks for letting me know. I'm rather busy this week so I will take a look at this as well as push an NPM update over the weekend.

SirAnthony commented 9 years ago

It is fine from git, please update npm package.

hellboy81 commented 9 years ago

Can node-diskusage version 0.1.3 work with node 0.12 ?

jduncanator commented 9 years ago

@hellboy81 When I get around to releasing an update it will work on 0.12. I'm working on a few changes to library before I release the next version

hellboy81 commented 9 years ago

Hmm.. Which incompatibilities has the current (0.13) and the previous library version with with node 0.12.7? I can compile on my Linux machine using nvm with 0.12.7

hellboy81 commented 9 years ago

As I mentioned the node-ffi since version 1.3.0 / 2015-03-22 supports 0.12.x:

  • add support for io.js >= v1.1.0 and node.js v0.12.x via na
hellboy81 commented 9 years ago

We are still waiting for 0.12 compatible version...

hellboy81 commented 9 years ago

This library can be replaced with diskspace.js

jduncanator commented 9 years ago

Sorry for the huge delays. I've setup automatic deployment now so it shouldn't happen again in the future. Version 0.1.4 is now available on npm.

As the API changed in a non-breaking manner I decided not to bump to version 0.2.x and instead went to 0.1.4.

starquake commented 9 years ago

Thanks! It works! I prefer your use of statvfs and GetDiskFreeSpaceEx over parsing the output of a command line tool.