Fix npm#1491 Don't try to mkdir('/')

buzypi · Oct 4, 2011 · b240b93 · b240b93
1 parent 87d3689
commit b240b93
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/lib/utils/mkdir-p.js b/lib/utils/mkdir-p.js
@@ -27,6 +27,12 @@ function mkdir (ensure, mode, uid, gid, cb_) {
 
   ensure = path.resolve(ensure).replace(/\/+$/, '')
 
+  // mkdir("/") should not do anything, since that always exists.
+  if (!ensure
+      || (process.platform === "win32" && p.match(/^[a-zA-Z]:(\\|\/)?$/))) {
+    return cb_()
+  }
+
   if (mkdirCache.hasOwnProperty(ensure)) {
     return mkdirCache[ensure].push(cb_)
   }