Open felix9743 opened 1 year ago
I've confirmed that this is indeed an issue. Thanks!
I've been looking into this issue, and found lots of interesting questions:
globwalk
crate used by sic
, or a bug or unsupported expansion in globwalk
glob
(unmaintained) and nu_glob
(maintained) support a smaller subset than most othersglobwalk
, the one used by this crate combines globset
via ignore
(as alternative to using globset
and walkdir
directly), ignore
can add glob overrides which affect directory traversalwax
supports an extensive set of syntax, and is very well tested, but only supports forward slashes, and flatly rejects drive letters (for windows), requiring some pre-processing and post-processing in the name of cross-platform support ..
(directory up) and .
(current directory), or drive letters (windows drive mounting points)
./path
relative paths in the current codebasehello/../world/*.png
or hello/./world/*.png
, and what should hello//world
mean?.
(current directory) beyond the first path component (first path component: ./some
, beyond: hello/./some
), then you may run into issues like #1656 globset
and a libraries which also resolves actual file system paths, like globwalk
and glob
..
and .
?globwalk
is unmaintained
sic
to the globwalker
crateI'm also still considering dropping absolute path support; then a pre-processed version of wax
may be preferable (like I've implemented here)
Still also happy to use/replace the glob library used. A list of libraries I'm aware of follows:
No filesystem walker
Includes filesystem walker
I use the Windows binary to convert the images extracted from a PDF but when I give a relative path, it tells me that it does not find it.
It works properly with
--input
, so the problem probably comes from--glob-input
.It however works correctly if I give an
absolute path
.