Skip to content
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

Add ability to limit files considered based on min/max file size #33

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

clara-j
Copy link

@clara-j clara-j commented Sep 13, 2015

I have added b and B options to limit the size of files considered given a numeric value for each, for the number of KB

@seb-mueller
Copy link

Are there any plans to merge those changes in at some point? The -b and -B options seem very useful.

@seb-mueller
Copy link

Thanks for pointing this out. I wasn't aware of jdupes and just tried it out. -X works like a charm.

@eMPee584
Copy link

eMPee584 commented Apr 3, 2019

Dear @adrianlopezroche: why did you censor this helpful comment (23/11/2017, 14:21) by @jbruchon about jdupes?

It is unlikely that this will be merged as-is, especially since it now has merge conflicts and the pull request has changes unrelated to the addition of the feature. It's also two years old. If you want this feature now, I have it as -x [+]size (and soon to be moved to -X size[+-=]:number[suffix]) in my fork. In master -X is already available which has the side effect of allowing multiple -x arguments to work as expected.

@jbruchon
Copy link

jbruchon commented Apr 3, 2019

Dear @adrianlopezroche: why did you censor this helpful comment (23/11/2017, 14:21) by @jbruchon about jdupes?

I deleted all of my posts on this repo. Please do not bother Adrian about my posts again. Thank you.

@FabioPedretti
Copy link

This was implemented in 2.0.0:
https://github.com/adrianlopezroche/fdupes/releases/tag/2.0.0
So, consider closing this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants