JSRocksHQ / harmonic

The next static site generator
http://harmonicjs.com/
MIT License
282 stars 26 forks source link

child_process: customFds option is deprecated, use stdio instead #91

Closed jaydson closed 9 years ago

jaydson commented 9 years ago

Every time i run harmonic i get this message.

UltCombo commented 9 years ago

Me too. It probably originates from a dependency. Will investigate this once I publish a working slush-es6 version.

UltCombo commented 9 years ago

This is an issue with v8-argv, here's the source.

Reported issue: logicalparadox/v8-argv#3

jaydson commented 9 years ago

Great. I saw your PR, let's wait for the approval on v8-argv project? I mean, now we're transpiling the code. Is v8-argv yet necessary? Questions questions :p

UltCombo commented 9 years ago

We're using Node's native generators, that's why we need v8-argv to add the --harmony flag.

jaydson commented 9 years ago

Yes, but we're using native generators because i started this way. My point here is we're transpiling everything, and generators can be transpiled too, and we don't need to force the "user" to have 0.11.* node env.

PS1: I don't think is bad thing to have node 0.11.* as requirement PS2: For sure native generators are better than transpiled

UltCombo commented 9 years ago

Thing is, even if we transpile generators, we won't be able to use co and co-dependent libraries unless they were transpiled too.

Right now, Harmonic depends on co-prompt, so transpiling only the Harmonic code wouldn't work in older Node versions as co-prompt uses native generators.

jaydson commented 9 years ago

Hey, this issue was solved by one of your commits. It was on purpose or an accident? Anyway, good job :+1:

UltCombo commented 9 years ago

On purpose, yes -- #107 removed the v8-argv dependency which was causing the issue. I forgot to write it in the PR, then I reminded about it when I was away from the computer and ended up forgetting again hah. Thanks for confirming that it is fixed. :+1:

jaydson commented 9 years ago

great!