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

Merged
merged 1 commit into from Dec 14, 2011

Conversation

Projects
None yet
3 participants
@papandreou
Contributor

papandreou commented Dec 14, 2011

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

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

glejeune added a commit that referenced this pull request Dec 14, 2011

Merge pull request #4 from papandreou/patch-1
graphviz.js: Don't require the 'sys' module.

@glejeune glejeune merged commit dece727 into glejeune:master Dec 14, 2011

@papandreou

This comment has been minimized.

Show comment
Hide comment
@papandreou

papandreou Dec 16, 2011

Contributor

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

Contributor

papandreou commented Dec 16, 2011

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

@glejeune

This comment has been minimized.

Show comment
Hide comment
@glejeune

glejeune Dec 16, 2011

Owner

Soon ;)

Owner

glejeune commented Dec 16, 2011

Soon ;)

@jeffbski

This comment has been minimized.

Show comment
Hide comment
@jeffbski

jeffbski Mar 13, 2012

+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!

+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

This comment has been minimized.

Show comment
Hide comment
@glejeune

glejeune Mar 19, 2012

Owner

Done...

Owner

glejeune commented Mar 19, 2012

Done...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment