search

jbenet / node-subcomandante

taaaaake ooooooon meeee
4 stars 8 forks source link

Use fork instead of process.argv[0] #3

Closed dignifiedquire closed 8 years ago

dignifiedquire commented 8 years ago

Closes #2

There might be a better way to do this, but this effectively a clone of comandate in fork.js which forks instead of spawns.

jbenet commented 8 years ago

may want to publish fork as a module.

this LGTM.

we should probably have tests. i have a couple i pushed to #4

jbenet commented 8 years ago

thanks @Dignifiedquire

jbenet commented 8 years ago

@Dignifiedquire want to try this version out in station before i publish it as a patch update?

dignifiedquire commented 8 years ago

I already tested it before I created the PR, station was finally running as a packaged instanced as soon as I applied it :)

daviddias commented 8 years ago

woot! :)

jbenet commented 8 years ago

Ok, published as 1.0.5

dignifiedquire commented 8 years ago

@jbenet thanks