graphviz.js: Don't require the 'sys' module.

glejeune / node-graphviz

Node.js interface to the GraphViz graphing tool

http://algorithmique.net

MIT License

197 stars 58 forks source link

graphviz.js: Don't require the 'sys' module. #4

Closed papandreou closed 12 years ago

papandreou commented 12 years ago

It's not used and it causes a deprecation warning with node.js 0.6 because the module has been renamed to 'util'.

papandreou commented 12 years ago

Thanks for merging :). How about a new release to the npm repo?

glejeune commented 12 years ago

Soon ;)

jeffbski commented 12 years ago

+1 Can you push the version 0.0.6 that doesn't require('sys') to npm? It appears that you have already done the work, just needs to be pushed to npm to be available.

Thanks!

glejeune commented 12 years ago

Done...