Re: [PATCH] Use find ... | xargs rm -f instead of find ... -delete

[David Kastrup]

Graham Percival <address@hidden> writes:

>> Then change it to something like this:
>> 
>>      find $(outdir)/offline-root -type l -print0 | xargs -0 rm -f --
>> 
>> Or even this (completely POSIX compliant):
>> 
>>      find $(outdir)/offline-root -type -l -exec rm -f -- '{}' +
>> 
>> (but then I've to patch it again, since find(1) on OpenBSD doesn't yet
>> support the `-exec command {} +' primary).
>
> Well, if that's part of POSIX, then adding it can't be a bad
> thing.  :)

Isn't AFAIK.

> But for the purposes of the lilypond build scripts, I don't think it's
> necessary.  The first version of the patch should be fine.

Ok, if the directory is complete under user control (not world-writable
in general: if it were, you could just put nasties in the Makefiles
anyway).

-- 
David Kastrup