mklabs / node-build-script

MIT License
322 stars 55 forks source link

CSS task generates empty files #39

Closed ottok closed 12 years ago

ottok commented 12 years ago

I just updated to the newest node-build-script and the CSS task broke. It generates only empty files:

Running "css:css/libraryproject.css" (css) task Uncompressed size: 0 bytes. Compressed size: 0 bytes gzipped (0 bytes minified). Writing css files to css/libraryproject.css...OK

mklabs commented 12 years ago

Hey ottok,

Thanks for the report. That's a bit weird.. I didn't run into this issue myself. This is is pretty basic, but we have some test on the css task, to make sure everything works properly.

I'll need to investigate. Can you provide a reproducible test case?

Also a snippet of your gruntfile may help me, the task have been tested with this basic configuration:

css: {
  'css/style.css': ['css/style.css']
},
ottok commented 12 years ago

Test case: git clone git@github.com:Seravo/Library-Directory.git sudo npm install -g connect express hogan grunt https://github.com/h5bp/node-build-script/tarball/master cd Library-Directory h5bp

-> CSS fails.

But with "sudo npm install -g https://github.com/h5bp/node-build-script/tarball/998a7ebea4369e042ea22a0b482a5d4dcc93a364" building CSS works (most of the time, still it randomly omits style.css from result compress.css).

mklabs commented 12 years ago

Thanks a lot for this. It'll really help me understanding what's going on.

I'll try to get it fixed asap, but I probably won't be able to do so up to this week-end.

mklabs commented 12 years ago

Wrong assumption on destination file being the same than one of the source file, was broken whenever the destination file does not exists yet.

It should be ok now. Please test this and re-open if it's not.

ottok commented 12 years ago

I confirm that this too works now. Thanks!