[0.6.0] Throws exception when using node cli via child_process

[brentlintner]

Steps to Reproduce:

git clone git@github.com:blackberry-webworks/Ripple-UI.git 
git checkout 3561ad5a755232c3e0a8ee01a11736a9dedfce69
./configure
npm install -g csslint@0.6.0
jake lint

Expected: CSSLint to run properly

Actual: Crashes with this trace:

node.js:134
    throw e; // process.nextTick error, or 'error' event on first tick
    ^
Error: ENOTSUP, Operation not supported
    at Object.fsyncSync (fs.js:352:18)
    at Object.print (/Users/brent/local/node/lib/node_modules/csslint/cli.js:208:12)
    at processFile (/Users/brent/local/node/lib/node_modules/csslint/cli.js:73:17)
    at /Users/brent/local/node/lib/node_modules/csslint/cli.js:129:36
    at Array.forEach (native)
    at processFiles (/Users/brent/local/node/lib/node_modules/csslint/cli.js:127:23)
    at cli (/Users/brent/local/node/lib/node_modules/csslint/cli.js:193:14)
    at Object.<anonymous> (/Users/brent/local/node/lib/node_modules/csslint/cli.js:203:1)
    at Module._compile (module.js:402:26)
    at Object..js (module.js:408:10)

Note: The CLI is being executed via the child_process module and that is where it seems to fail. Running it straight in the terminal works as expected, so I have not ruled out the possibility it is still our code causing the issue. I plan to look into it more when I can, but it seems like a CSSLint specific bug (as everything works in 0.5.0).

System

• OSX 10.6.8
• Node v0.4.11, NPM v1.0.22

[brentlintner]

Oops, FYI I updated the steps to reproduce (missed something).

[eriwen]

I'm having this same issue (as of 0.6.0) with Gradle, and have traced it to the use of "fs.fsyncSync()" in the cli/node.js file within print().

[nzakas]

Interesting. I made that change because Node.js uses stream printing for stdout, so capturing stdout via > wasn't working correctly when using console.log (see #175). This seemed to work, but apparently not in all cases.

[nzakas]

Related to Node.js bug: nodejs/node-v0.x-archive#1669

[eriwen]

Wanted to say thanks for pushing out a fix for this so quickly.

[brentlintner]

Sweet, kudos on the workaround. Many thanks!

[hakta]

I'm getting same error message:

Steps:

npm install csslint@0.6.1
node_modules/.bin/csslint

Error message:

csslint: No files specified.

node.js:208
        throw e; // process.nextTick error, or 'error' event on first tick
              ^
TypeError: Object #<WriteStream> has no method 'flush'
    at Object.quit (/home/vpoluceno/labs/ritty/node_modules/csslint/cli.js:213:38)
    at cli (/home/vpoluceno/labs/ritty/node_modules/csslint/cli.js:193:9)
    at Object.<anonymous> (/home/vpoluceno/labs/ritty/node_modules/csslint/cli.js:203:1)
    at Module._compile (module.js:425:26)
    at Object..js (module.js:443:10)
    at Module.load (module.js:344:31)
    at Function._load (module.js:303:12)
    at Array.<anonymous> (module.js:463:10)
    at EventEmitter._tickCallback (node.js:200:26)

System:

nodejs: v0.5.7
npm: 1.0.30
ubuntu: 11.04
uname -a: Linux xxxx-xxxx 2.6.38-11-generic-pae #50-Ubuntu SMP Mon Sep 12 22:21:04 UTC 2011 i686 i686 i386 GNU/Linux

[nzakas]

@hakta - This looks like a different error, can you please open a separate ticket?

[hakta]

@nzakas - Sure, new ticket here #190