Adding filtering to avoid some file analysis about kolekto HOT 5 CLOSED

You can do:

$ kolekto import ./Folder/*.mkv

Is this feature really needed?

from kolekto.

Yes it can be a solution but not the best in my opinion as we need to run
the import command for each extension. Unless we can do something like
this :

$ kolekto import ./Folder/*.{mkv,avi,webm}

I'll try to implement this feature and make a PR for it if you want :-) it
will be my small contribution to this great tool.
Le 2 août 2013 23:06, "Antoine Millet" [email protected] a écrit :

You can do:

$ kolekto import ./Folder/*.mkv

Is this feature really needed?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/12#issuecomment-22035824
.

from kolekto.

Actually your example will work as expected (your shell will expand the pattern) :-).

from kolekto.

Ok so maybe this feature is not needed as you said or with a low priority
to give the ability to import a folder without thinking about what is
inside it.

from kolekto.

Note that I'm not totally against your idea, I just want to be sure that the feature is really useful.

I also think that this feature is not so trivial to implement. A naive filtering based on the file extension is not sufficient (imho), the number of different video file extensions is very large, and if we miss one of them, it can be annoying to a user. Adding a parameter to enable the filter and specify extension doesn't bring more functionally than the shell expansion. A filter based on the file content (like the "file" binary) become really complicated...

from kolekto.