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.

Sign up for GitHub

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

Jump to bottom

'distclean' requires 3 traversals of project tree #36

Closed
pprindeville opened this issue Dec 14, 2012 · 3 comments
Closed

'distclean' requires 3 traversals of project tree #36

pprindeville opened this issue Dec 14, 2012 · 3 comments
Assignees
@aleks-f
Labels
enhancement
Milestone
Release 1.5.1

Comments

@pprindeville
Copy link
Contributor

The sequence:

   find $(POCO_BUILD) -name obj -type d -print0 | xargs -0 rm -rf
   find $(POCO_BUILD) -name .dep -type d -print0 | xargs -0 rm -rf
   find $(POCO_BUILD) -name bin -type d -print0 | xargs -0 rm -rf

can be refactored as:

   find $(POCO_BUILD) \( -name obj -o -name .dep -o -name bin \) -type d -print0 | xargs -0 rm -rf

and do the same in a single traversal.
@pprindeville
Copy link
Contributor Author

Should also point out that POSIX 'find' supports parens as standard:

http://pubs.opengroup.org/onlinepubs/009695399/utilities/find.html

@ghost ghost assigned aleks-f Dec 14, 2012
@aleks-f
Copy link
Member

aleks-f commented Dec 14, 2012

will be in 1.6.0

aleks-f added a commit that referenced this issue Jan 6, 2013
@aleks-f
GH #36: 'distclean' requires 3 traversals of project tree
574be1a
@aleks-f
Copy link
Member

aleks-f commented Jan 6, 2013

added to develop for 1.5.1:

574be1a

@aleks-f aleks-f closed this as completed Jan 6, 2013
obiltschnig pushed a commit that referenced this issue Jan 12, 2017
@mkrivos
Merge pull request #36 from pocoproject/develop
42b89e0 
Sync 3.1.2017
@bakkiraju bakkiraju mentioned this issue Sep 12, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aleks-f aleks-f

Labels
enhancement
Projects
None yet
Milestone
Release 1.5.1
Development

No branches or pull requests
2 participants
@pprindeville @aleks-f