use isFIFO() instead to more narrowly target <() usage

browserify · Mar 29, 2013 · 790cdf5 · 790cdf5
1 parent c396065
commit 790cdf5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/async.js b/lib/async.js
@@ -15,7 +15,7 @@ module.exports = function resolve (x, opts, cb) {
         fs.stat(file, function (err, stat) {
             if (err && err.code === 'ENOENT') cb(null, false)
             else if (err) cb(err)
-            else cb(null, !stat.isDirectory())
+            else cb(null, stat.isFile() || stat.isFIFO())
         });
     };
     var readFile = opts.readFile || fs.readFile;

diff --git a/lib/sync.js b/lib/sync.js
@@ -1,14 +1,15 @@
 var core = require('./core');
 var fs = require('fs');
 var path = require('path');
-var existsSync = fs.existsSync || path.existsSync;
 
 module.exports = function (x, opts) {
     if (core[x]) return x;
 
     if (!opts) opts = {};
     var isFile = opts.isFile || function (file) {
-        return existsSync(file) && !fs.statSync(file).isDirectory()
+        try { var stat = fs.statSync(file) }
+        catch (err) { if (err && err.code === 'ENOENT') return false }
+        return stat.isFile() || stat.isFIFO();
     };
     var readFileSync = opts.readFileSync || fs.readFileSync;