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

Already on GitHub? Sign in to your account

Seems like a pretty simple fix, did the trick. #120

Merged
merged 1 commit into from Sep 24, 2012

Conversation

Projects
None yet
2 participants
Contributor

jacobSingh commented Sep 2, 2012

Docs could also use something specifying that this will not work with standard getopt syntax, -mN does not work, only -m N. Similarly --max=N will not work.

Using the node getopt library seems like it would be useful here, no?

Contributor

chriso commented Sep 24, 2012

Thanks for this. I implemented the opts parsing part of node.io before there were any libraries available to do it for you. I plan on bundling a dep in v2 which will use the standard getopt syntax.

@chriso chriso closed this Sep 24, 2012

@chriso chriso reopened this Sep 24, 2012

@chriso chriso added a commit that referenced this pull request Sep 24, 2012

@chriso chriso Merge pull request #120 from jacobSingh/master
Seems like a pretty simple fix, did the trick.
5e715f5

@chriso chriso merged commit 5e715f5 into node-js-libs:master Sep 24, 2012

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