giggio / globdir

A library that enables you to glob directories with .NET.
6 stars 3 forks source link

Globbing requires Unix path separators. #1

Open nemec opened 11 years ago

nemec commented 11 years ago

The documentation should specify that globbing doesn't work unless the path uses forward slashes for directory separation, especially since the tagline reads "A way to glob Windows directories" which leads me to believe typical Windows paths will be accepted.

That, or accept both path separators when globbing.

giggio commented 11 years ago

Agreed. It is open to anyone who wishes to implement and send a pull request. You may check the original implementation, which considered this at: https://github.com/IronLanguages/main/blob/master/Languages/Ruby/Ruby/Builtins/Glob.cs If you make it work on unix, just send me the pull request.

atifaziz commented 10 years ago

Commit 07f5cec adds backslash support on Window but only when no escaping is requested, which is one of default flags. When escaping is requested then backslash causes conflict and Unix directory separator of slash must be used instead.