-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
glob patterns like "*/z1.txt" don't work (ie when a glob happens before / it seems) #29
Comments
Closes #29 ``` foo/a.txt bar/a.txt baz/a.txt ``` ```nim import glob import sequtils echo toSeq(walkGlob("*/a.txt")) # -> @["foo/a.txt", "bar/a.txt", "baz/a.txt"] ```
@citycide can we reopen? still doesn't work properly:
import glob,sequtils
echo toSeq(walkGlob("*/*/z1.txt"))
@[] |
@citycide friendly ping :) this bug really limits usability of this otherwise excellent tool ... |
Yeah I'm working on it but it hasn't been as simple as I thought it would be. Still in progress. |
@citycide can you adapt https://github.com/git/git/blob/master/wildmatch.c to Nim (either via importc or via manual conversion, maybe starting gradually, starting from importc)? that seems like exactly what we want top-level glob can use it as a building block; and we can implement BFS/DFS search on top of that. happy to discuss implementation details; but I think it could make sense to start from |
@timotheecour that doesn't support extended glob features like |
Exactly! |
/cc @citycide this bug seems serious:
on linux:
find
pwd
/tmp/d03
/tmp/d03/A1
/tmp/d03/A1/z2.txt
/tmp/d03/A1/Z2.txt
/tmp/d03/a1
/tmp/d03/a1/z1.txt
The text was updated successfully, but these errors were encountered: