[pacman-dev] [PATCH 2/2] Allow wildcards in PURGE_TARGETS to match any type of file except for directories.

[Jeremy Huntwork]

On Saturday, June 2, 2012 at 9:16 PM, Allan McRae wrote:
> > scripts/makepkg.sh.in (http://makepkg.sh.in) | 2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/scripts/makepkg.sh.in (http://makepkg.sh.in) b/scripts/makepkg.sh.in (http://makepkg.sh.in)
> > index b0918ae..eb239e3 100644
> > --- a/scripts/makepkg.sh.in (http://makepkg.sh.in)
> > +++ b/scripts/makepkg.sh.in (http://makepkg.sh.in)
> > @@ -1048,7 +1048,7 @@ tidy_install() {
> > local pt
> > for pt in "${PURGE_TARGETS[@]}"; do
> > if [[ ${pt} = "${pt//\/}" ]]; then
> > - find . -type f -name "${pt}" -exec rm -f -- '{}' \;
> > + find . ! -type d -name "${pt}" -exec rm -f -- '{}' \;
> > else
> > rm -f ${pt}
> > fi
> 

Sorry for bringing this up after the fact, but with this change is there any reason to execute that if test? Why not just use the find command for all given purge targets?

Thanks,

JH