From my point view, adding extra command line options is not a problem. If people are going to use such a feature it would be better for them to do it via Geeqie rather than via another project.

Although, if implemented, there will be a configuration option to disable it, I am not sure about the implications of making python a dependency. Perhaps @gusnan has an opinion.

The only python dependency in the project so far is involved with running tests, so is not part of the distribution.

Sorry, I should have pointed out that the Python part is completely optional. Basically how the main program works is that it runs some command once for each group of duplicates (passing them as arguments). By default the command is just echo but you can specify anything you like.

I just happened to use Python to create a script that accepts the filenames as parameters, and if their names match certain patterns (in my case this hints on their source) deletes smaller files without asking for confirmation.

FTR, here is the standalone version. I'll try to come up with a PR against geeqie soon. I understand you'd prefer to make this a mode of geeqie itself, rather than a seperate executable? 🤔

. I understand you'd prefer to make this a mode of geeqie itself, rather than a seperate executable?

It is your choice if you want a separate project/executable. I think it would be useful if it were available within Geeqie.

main.cc and remote.cc are probably the only files affected.

Please note CODING.md, but ignore the bits that everyone else ignores.
When you make a pull request to Geeqie on GitHub, static tests are automatically run.
You can run the tests locally first by ./scripts/test-all.sh

The Help file/User Manual will need an update. I can do that.

[I have noticed that very occasionally the similarity algorithm gets it completely wrong. I am not sure I would use an automatic delete without a visual check. But everyone has a choice....]

. I understand you'd prefer to make this a mode of geeqie itself, rather than a seperate executable?

It is your choice if you want a separate project/executable. I think it would be useful if it were available within Geeqie.

I agree about integrating it in the Geeqie project. I was asking about whether it should be an option within /usr/bin/geeqie or a separate executable (e.g. /usr/bin/geeqie-duplicate-finder or such)?

Ah, sorry.
I was assuming an additional option:

geeqie --remote --duplicates-program=<prog name> --duplicates-threshold=<n> --duplicates-files=<file list>
geeqie -r -p <prog name> -t <n> -m <file list>

Maybe the -m option is not necessary - just take whatever file list is appended on the command line.

Geeqie now has bash command line completion which makes things easier with long-options.

[I am working on making Geeqie a GtkAppliction, which will make command line processing easier. The --remote option will disappear]

Keeping everything together makes distribution easier.

@caclark I was really hoping it would be implemented this way.

Maybe the -m option is not necessary - just take whatever file list is appended on the command line.

I having trouble figuring out how to get access to this "remainder" file list from within the callback I added for --duplicates-program 🤔
Can you maybe provide some hints?

The command line processing is a mess...

The attached diff might help you to get something to work in some way or other, but it is not really suitable to put in the repo.

Run ./build/src/geeqie from one terminal window, and from a second terminal window run:
./build/src/geeqie -r <some file list>
./build/src/geeqie -r -m

I will work on making the command line processing more logical.

remote-duplicates-1.diff.gz