"**/" pattern doesn't match "./" path part

[cowboy]

This seems like a bug to me, intuitively, but I don't see it addressed in the docs anywhere:

var minimatch = require("minimatch");

minimatch("foo/bar.js", "**/foo/**")     // true
minimatch("./foo/bar.js", "./**/foo/**") // true
minimatch("./foo/bar.js", "**/foo/**")   // false

Shouldn't the last example return true because **/ should match ./?

Ref. cowboy/node-globule#11

[corbanbrook]

This may be related to an issue i am having trying to match a file in a hidden dir.

minimatch("/Users/corban/.go/src/app.js", "**/app.js") // false

[Rich-Harris]

@corbanbrook I had a similar issue - #51 - and solved it by passing {dot: true} as the third argument to minimatch():

minimatch("/Users/corban/.go/src/app.js", "**/app.js", {dot: true}) // true

This doesn't fix @cowboy's case though:

minimatch("./foo/bar.js", "**/foo/**", {dot: true}) // false

[implico]

So, why don't we fix this? This also breaks such packages as gulp-filter functionality, also with patterns containing "../", what is sometimes quite essential.

[cb1kenobi]

+1. It would really be nice if minimatch supported .. so that gulp-filter worked. It just came up again the other day: sindresorhus/gulp-filter#67 (comment).

[Lucas-C]

👍

Actually hidden files NEVER match :

var minimatch = require('minimatch').Minimatch;

var matcher = minimatch('**');

console.log(matcher.match('bob')); // true
console.log(matcher.match('.bob')); // false

I tested this behaviour with minimatch v0.3.0, v2.0.7 & v3.0.2

[Lucas-C]

And yes, this affects any file with a directory starting with a dot anywhere in its full path.
E.g. /home/bob/.jenkins/jobs/HellowWorld/workspace/snafu.js will NEVER match any **-based pattern.

I think this is a major issue.

[Lucas-C]

Please disregard my previous comments: I found out about {dot: true}.

I think this really shoud be the default behaviour though.