Chmod bins in non-global folders, too

npm · Apr 25, 2011 · 922e99f · 922e99f
1 parent 41c285e
commit 922e99f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/build.js b/lib/build.js
@@ -101,16 +101,16 @@ function linkBins (pkg, folder, parent, gtop, cb) {
   }
   var binRoot = gtop ? path.resolve(npm.config.get("prefix"), "bin")
                      : path.resolve(parent, ".bin")
-  log.verbose([pkg.bin, binRoot], "bins linking")
+  log.verbose([pkg.bin, binRoot, gtop], "bins linking")
   asyncMap(Object.keys(pkg.bin), function (b, cb) {
     linkIfExists(path.resolve(folder, pkg.bin[b])
                 ,path.resolve(binRoot, b)
                 ,gtop && folder
                 ,function (er) {
-      if (er || !gtop) return cb(er)
+      if (er) return cb(er)
       // bins should always be executable.
       fs.chmod(path.resolve(folder, pkg.bin[b]), 0755, function (er) {
-        if (er) return cb(er)
+        if (er || !gtop) return cb(er)
         output.write(path.resolve(binRoot, b)+" -> "
                     +path.resolve(folder, pkg.bin[b]), cb)
       })