ADD node_path configuration item support.

[chigix]

Signed-off-by: Richard Lea chigix@zoho.com

[marijnh]

Isn't directly overriding tern_command just as effective with less moving parts?

[chigix]

No, I think they are not same from the perspective of design.

• node_path is just for users indicating the binary file only, without any particular or ambiguous user input arguments, which is also with potential risky for unexpected problem in the future.
• It is no need for users to learn and check the suggested arguments and options to node command for running ternjs, which is perfectly made in your plugin_load script.
• tern_command is really an absolute configuration for allowing users, expecially some advanced users, to make some special configuration with their own arguments, which is not common and always not general arguments, of course I think.
• At last this binary providing is totally optional, none of any problem would occured related to compatibility and comprehensiblity. But This configuration item support is really useful as simple.

So, maybe different view would be there, but this configuration added should be treated as nice to have, at least in my opinion.

[marijnh]

I think the interaction between this and tern_command is going to be too confusing (it only takes effect if you don't specify tern_command), so I'm going to decline.