Error: EISDIR: illegal operation on a directory about deadsimple-less-watch-compiler HOT 3 CLOSED

@serjp which version of less-watch-compiler Could you try the latest?

from deadsimple-less-watch-compiler.

@jonycheung [email protected], the latest in npm. Tried version 1.6.13 - seems to be ok.

from deadsimple-less-watch-compiler.

I am experiencing the same problem with every version > 1.4 (1.4.2 works ok).

Just set the watchFolder to be relative:

./less
../src/less (in my case)

and the exception is thrown.

Debugging the code of the latest version (1.10) I encountered the following code in lessWatchCompilerUtils.js:184:

fileWatcher: function fileWatcher(f, files, options, filelist, fileimportlist, watchCallback) {
  if (filelist.indexOf(f) !== -1) return;
  filelist[filelist.length] = f;

  if (f.indexOf('.') !== -1) {
    fileimportlist[f] = fileSearch.findLessImportsInFile(f);
  }
  lessWatchCompilerUtilsModule.setupWatcher(f, files, options, watchCallback);
  for (var i in fileimportlist[f]) {
    if (filelist.indexOf(fileimportlist[f][i]) === -1) {
      lessWatchCompilerUtilsModule.setupWatcher(path.normalize(path.dirname(f) + '/' + fileimportlist[f][i]), files, options, watchCallback);
    }
  }
}

this function is called (probably the first time) with f set to the watchFolder and the second if (if (f.indexOf('.') !== -1)) will resolve to true if f has . inside it (and every relative folder name does) calling fileSearch.findLessImportsInFile(f) which will fail by calling fs.readFileSync(f, 'utf8') in filesearch.js:15.

The condition if (f.indexOf('.') !== -1) if pretty fragile one. Why not checking if f is a directory?

Changing the condition to if (f.lastIndexOf('.') > 2) solves the problem for me, but it's a hack.

Thank you.

from deadsimple-less-watch-compiler.