Skip to content

more simple rm pyc files command #698

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

Merged
merged 1 commit into from
Apr 20, 2016
Merged

more simple rm pyc files command #698

merged 1 commit into from
Apr 20, 2016

Conversation

gruzovator
Copy link
Contributor

No description provided.

@kennethreitz
Copy link
Contributor

I haven't seen this one used nearly as often. It's been a number of years since I investigated why, but perhaps it was because of cross-platform compatibility?

@gruzovator
Copy link
Contributor Author

I can't find any sign of compatibility problems with -delete action of 'find'. GNU Findutils doc recommends '-delete' to delete files:
https://www.gnu.org/software/findutils/manual/html_node/find_html/Deleting-Files.html

@sigmavirus24
Copy link
Contributor

I haven't seen this one used nearly as often.

I have, but perhaps I use find more often?

It's been a number of years since I investigated why, but perhaps it was because of cross-platform compatibility?

My guess would simply be that -delete is slightly newer and the older was just handed down to you.

@tylerdave
Copy link
Contributor

-delete was added by BSD and has been in GNU find for quite a while now. There are still systems out there w/ older version of find that don't have the -delete action but I honestly doubt very many of them are being used for development at this point.

@kennethreitz
Copy link
Contributor

Perfect, that's the answer I was looking for. I know at one point in my development career, '-delete' didn't work on OS X.

Thanks!

@kennethreitz kennethreitz merged commit e6d51a1 into realpython:master Apr 20, 2016
@sigmavirus24
Copy link
Contributor

I know at one point in my development career, '-delete' didn't work on OS X.

Good thing no one actually uses OS X ;)

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.

4 participants