actually use pathspecs. enable pathspec.create

maxkorp · maxkorp · commit 5afb809c594b · 2015-03-09T15:26:49.000-07:00
diff --git a/generate/input/descriptor.json b/generate/input/descriptor.json
@@ -1182,9 +1182,6 @@
       "functions": {
         "git_pathspec_match_list_free": {
           "ignore": true
-        },
-        "git_pathspec_new": {
-          "ignore": true
         }
       }
     },
diff --git a/lib/index.js b/lib/index.js
@@ -2,6 +2,7 @@ var NodeGit = require("../");
 
 var Index = NodeGit.Index;
 var Status = NodeGit.Status;
+var Pathspec = NodeGit.Pathspec;
 var Promise = require("nodegit-promise");
 /**
  * Return an array of the entries in this index.
@@ -25,15 +26,24 @@ Index.prototype.addAll = function(pathspec, flags, matchedCallback) {
   // even if it has no changes. Remove this when it's fixed in libgit2
   // https://github.com/libgit2/libgit2/issues/2687
   var paths = [];
+  var repo = this.owner();
   var statusCB = function(path) {
     paths.push(path);
   };
-  return Status.foreach(this.owner(), statusCB)
+  var idx = this;
+  return Pathspec.create(pathspec || "*")
+    .then(function(ps) {
+      pathspec = ps;
+      return Status.foreach(repo, statusCB);
+    })
     .then(function() {
       return Promise.resolve(paths);
     })
     .then(function(paths) {
-      return addAll.call(this, pathspec || "*", flags, matchedCallback, null);
+      paths = paths.filter(function(path) {
+        return !!(pathspec.matchesPath(0, path));
+      });
+      return addAll.call(idx, paths, flags, matchedCallback, null);
     });
 };
 
