clean/distclean speedup

Paul Fox pgf at brightstareng.com
Fri Mar 10 14:23:05 UTC 2006


 > Is there any policy about filename?   If a filename contained spaces
 >  the xargs would have problems.  You should add -print0 to the find
 >  command and -0 option to xargx.

i'd be really really surprised if files with spaces got into the
build, but you have a good point.  somone needs to add -print0 to
busybox find before this could be self-hosting, however.  (busybox
xargs has "-0", but it may not always be present.)

paul

 > 
 > Index: Makefile
 > ===================================================================
 > --- Makefile (revision 14459)
 > +++ Makefile (working copy)
 > @@ -498,18 +498,18 @@
 >       libbusybox.so* \
 >       .config.old busybox busybox_unstripped
 >   - rm -r -f _install testsuite/links
 > - - find . -name .\*.flags -exec rm -f {} \;
 > - - find . -name \*.o -exec rm -f {} \;
 > - - find . -name \*.om -exec rm -f {} \;
 > - - find . -name \*.os -exec rm -f {} \;
 > - - find . -name \*.osm -exec rm -f {} \;
 > - - find . -name \*.a -exec rm -f {} \;
 > + - find . -name .\*.flags \
 > +     -o -name \*.o \
 > +     -o -name \*.om \
 > +     -o -name \*.os \
 > +     -o -name \*.osm \
 > +     -o -name \*.a | xargs rm -f
 >  
 >  distclean: clean
 >   - $(MAKE) -C scripts/config clean
 >   - rm -f scripts/bb_mkdep
 >   - rm -r -f include/config $(DEP_INCLUDES)
 > - - find . -name .depend'*' -exec rm -f {} \;
 > + - find . -name .depend'*' | xargs rm -f 
 >   rm -f .config .config.old .config.cmd
 >  

=---------------------
 paul fox, pgf at brightstareng.com



More information about the busybox mailing list