Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updates for Node 0.6.x #3

Closed
wants to merge 1 commit into from
Closed

Updates for Node 0.6.x #3

wants to merge 1 commit into from

Conversation

chrisa
Copy link
Owner

@chrisa chrisa commented Nov 24, 2011

  • "util" for "sys".
    • Remove scope.Close() calls
    • Update wscript to cope with build/Release and build/default
    • Adjust paths to ease test-running.

@chrisa
Updates for Node 0.6.x
ecefeb9 
 * "util" for "sys".
 * Remove scope.Close() calls
 * Update wscript to cope with build/Release and build/default
 * Adjust paths to ease test-running.
@chrisa chrisa mentioned this pull request Nov 24, 2011
Open
@zamfi
Copy link

zamfi commented Feb 24, 2012

This branch fixes the issues I'm having with node-rsa under node 0.6.10. Is there a timeframe on integration?

@chrisa
Copy link
Owner Author

chrisa commented Feb 25, 2012

These changes should already be in master - I just ended up not merging them directly.

Please give v0.0.3 a go from npm, which should work on both 0.4.x and 0.6.x.

@chrisa chrisa closed this Feb 25, 2012
@zamfi
Copy link

zamfi commented Feb 25, 2012

Ah, indeed you have! That said, I needed to uninstall and reinstall the module... perhaps this fix merits a version number increase? (Or maybe there's some other mechanism to force npm to rebuild?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chrisa @zamfi