Directory Iteration and Recursion

[EmilianoJordan]

Is your feature request related to a problem? Please describe.
For some of my use cases it would be nice to be able to pass arguments/flags to iterate directories or recursively process images. This is, for the most part, for convenience. Most functionality can be done by the caller and and CLI. Python has glob, pathlib etc for accomplishing this from the caller. And the CLI could used something like this ls -R | grep '.jpeg\|.jpg' | xargs exif-stripper.

But it might still be worth implementing something in this package. I'd be happy to PR a full working example if you're open to that.

Describe the solution you'd like
I have a very rough proposal, The CLI would need updates to support the new function args but this should give an idea.

This does have some issues that will need to be addressed that I know of:

1. symlinks: I added recurse_symlinks: bool = False but the similar Path().glob() arg did not get added until python 3.13. I think this functionality should be included in this (>=3.8).
2. case sensitivity: very similar to symlinks but different.
3. file suffixes: There might be better strategies than the white list one I went with.

Thanks for this package in the first place!

[github-actions]

It looks like this is your first issue here – welcome! Please familiarize yourself with the contributing guidelines, if you haven't already.

[stefmolin]

Thanks for reaching out, @EmilianoJordan. Am I correct in assuming you don't use exif-stripper as a pre-commit hook? I ask because pre-commit has this functionality (and it also handles passing in only image files to the tool using https://github.com/pre-commit/identify):

pre-commit run --all-files  # process all images in the repository

In general, I'm not opposed to adding this to the tool itself, as long as it doesn't interfere with using the pre-commit hook.

[EmilianoJordan]

I did not know about identify, thanks for that!

You are correct, I have not found a pre-commit use case yet. I installed it as a CLI tool for now, and thought that I had a couple use cases for this + what I mentioned. Although I might be able to integrate exiftool for those, I also wanted the flexibility of having the same functionality in a python callable.

I'll put something together that doesn't break your pre-commit use case and see what you think.