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

2009-12-30 Thread Matthias Kilian
On Wed, Dec 30, 2009 at 09:59:51AM +0100, David Kastrup wrote: > >>> find $(outdir)/offline-root -type -l -exec rm -f -- '{}' + [...] > >> Well, if that's part of POSIX, then adding it can't be a bad > >> thing. :) > > > > Isn't AFAIK. > > The '+' thing appears to be POSIX if I am reading the G

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

2009-12-30 Thread David Kastrup
David Kastrup writes: > Graham Percival 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 -- '{}' + >>

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

2009-12-29 Thread David Kastrup
Matthias Kilian writes: > On Tue, Dec 29, 2009 at 08:16:37AM +0100, David Kastrup wrote: >> > - find $(outdir)/offline-root -type l -delete >> > + find $(outdir)/offline-root -type l | xargs rm -f >> > endif # ifeq ($(out),www) >> >> What's the rationale? > > The `-delete' primary isn't very

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

2009-12-29 Thread David Kastrup
Graham Percival 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

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

2009-12-29 Thread John Mandereau
Le mardi 29 décembre 2009 à 22:05 +0100, Matthias Kilian a écrit : > `must' is such a strong word. There are other GNUisms (or non-portable > extensions) hidden in the build system, like that `grep -L' at some > places. This will disappear in some future (when Texinfo has a good enough i18n suppor

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

2009-12-29 Thread Matthias Kilian
[this is becoming slightly offtopic, but anyway...] On Tue, Dec 29, 2009 at 08:19:57PM +, Graham Percival wrote: > > > > - find $(outdir)/offline-root -type l -delete > > > > + find $(outdir)/offline-root -type l | xargs rm -f [...] > > The `-delete' primary isn't very portable, it

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

2009-12-29 Thread John Mandereau
Le mardi 29 décembre 2009 à 20:19 +, Graham Percival a écrit : > On Tue, Dec 29, 2009 at 08:00:46PM +0100, Matthias Kilian wrote: > > The `-delete' primary isn't very portable, it's a GNU extension. > > IMHO, that's all we need to hear. -delete must be removed. Yes, that's what I figured ou

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

2009-12-29 Thread Graham Percival
On Tue, Dec 29, 2009 at 08:00:46PM +0100, Matthias Kilian wrote: > On Tue, Dec 29, 2009 at 08:16:37AM +0100, David Kastrup wrote: > > > - find $(outdir)/offline-root -type l -delete > > > + find $(outdir)/offline-root -type l | xargs rm -f > > > endif # ifeq ($(out),www) > > > > What's the ration

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

2009-12-29 Thread Matthias Kilian
On Tue, Dec 29, 2009 at 08:16:37AM +0100, David Kastrup wrote: > > - find $(outdir)/offline-root -type l -delete > > + find $(outdir)/offline-root -type l | xargs rm -f > > endif # ifeq ($(out),www) > > What's the rationale? The `-delete' primary isn't very portable, it's a GNU extension.

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

2009-12-28 Thread David Kastrup
Matthias Kilian writes: > --- > GNUmakefile.in |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/GNUmakefile.in b/GNUmakefile.in > index 6574e65..6c2c54f 100644 > --- a/GNUmakefile.in > +++ b/GNUmakefile.in > @@ -123,7 +123,7 @@ WWW-post: > $(buildscript-dir)/m

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

2009-12-28 Thread John Mandereau
Thanks, applied. John signature.asc Description: Ceci est une partie de message numériquement signée ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

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

2009-12-28 Thread Matthias Kilian
--- GNUmakefile.in |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/GNUmakefile.in b/GNUmakefile.in index 6574e65..6c2c54f 100644 --- a/GNUmakefile.in +++ b/GNUmakefile.in @@ -123,7 +123,7 @@ WWW-post: $(buildscript-dir)/mutopia-index -o $(outdir)/examples.html inp