Skip to content

more simple rm pyc files command#698

Merged
kennethreitz merged 1 commit into
realpython:masterfrom
gruzovator:master
Apr 20, 2016
Merged

more simple rm pyc files command#698
kennethreitz merged 1 commit into
realpython:masterfrom
gruzovator:master

Conversation

@gruzovator
Copy link
Copy Markdown
Contributor

No description provided.

@kennethreitz
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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