Use `path.relative` instead of manual string replacement.

Fixes Windows inconsistency (#26).
ryanmcgrath · May 14, 2012 · 01190602dac64924fca2dae11912ffb560e636a0 · 0119060
1 parent a4e5005
commit 01190602dac64924fca2dae11912ffb560e636a0
Unified Split

Showing with 2 additions and 2 deletions.

+2 −2 lib/wrench.js
diff --git a/lib/wrench.js b/lib/wrench.js
@@ -49,7 +49,7 @@ exports.readdirSyncRecursive = function(baseDir) {

    // convert absolute paths to relative
    var fileList = readdirSyncRecursive(baseDir).map(function(val){
-        return val.replace(baseDir + '/', '');
+        return _path.relative(baseDir, val);
    });

    return fileList;
@@ -106,7 +106,7 @@ exports.readdirRecursive = function(baseDir, fn) {

            fn(null, curFiles.map(function(val) {
                // convert absolute paths to relative
-                return val.replace(baseDir + '/', '');
+                return _path.relative(baseDir, val);
            }));

            if (waitCount == 0) {