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