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.

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

#910: add support for --node flag to pass arguments to node #945

Merged
1 commit merged into from
Dec 18, 2010

Conversation

michaelficarra
Copy link
Collaborator

Usage as follows: coffee --print -e --node "--debug -v --whatever" --lint -b

--node argument takes a string containing an argument list to be passed to node and can be placed anywhere in coffee's argument list.

Documentation will have to be written, I have neither the time nor the patience for that right now.

@satyr
Copy link
Collaborator

satyr commented Dec 17, 2010

Isn't it confusing with --nodes?

@michaelficarra
Copy link
Collaborator Author

Eh, the name of the argument doesn't matter to me. --node wouldn't confuse me, but it's an easy change. Have any suggestions?

@TrevorBurnham
Copy link
Collaborator

Well, -a/--arguments isn't taken.

@michaelficarra
Copy link
Collaborator Author

That doesn't exactly convey anything about how they're passed to node. arguments is definitely too general of a name.

@TrevorBurnham
Copy link
Collaborator

Right, but every single letter in the phrase "node options" is already taken as a single-char flag... and I thought the name of the argument didn't matter to you?

I guess you could call it --argsfornode. People will just use -a anyway.

@michaelficarra
Copy link
Collaborator Author

It doesn't, but if we're trying to pick one that's better than what we have, we should pick an appropriate one. How about --nodejs? --args-for-node isn't bad either. Really, though, it's arbitrary. It could be literally anything at all and I wouldn't have a problem with it.

@jashkenas
Copy link
Owner

Thanks for the patch, Michael. I've merged (a greatly simplified version of) it to master there:

bc4498e

Closing the ticket.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants