On 7/31/07, Jonas Karlsson <[EMAIL PROTECTED]> wrote:
> 2007/7/31, Jonas Karlsson <[EMAIL PROTECTED]>:
> > On Tue, 31 Jul 2007 07:01:51 +0200, Lucas C. Villa Real
> <[EMAIL PROTECTED]> wrote:
> >
> > > CVSROOT:      /sources/goboscripts
> > > Module name:  tools
> > > Changes by:   Lucas C. Villa Real <lucasvr>   07/07/31 05:01:51
> > >
> > > Modified files:
> > >       Compile/bin    : PackRecipe
> > >
> > > Log message:
> > >       Stay backwards compatible with other versions of xargs.
> > >
> > > CVSWeb URLs:
> > >
> http://cvs.savannah.gnu.org/viewcvs/tools/Compile/bin/PackRecipe?cvsroot=goboscripts&r1=1.7&r2=1.8
> > >
> > [...]
> > >        #remove backups
> > > -      find "$name/$version" | grep -E '~$' | xargs -i rm -f {}
> > > +      find "$name/$version" | grep -E '~$' | xargs rm -f {}
> >
> > I don't know for other versions but xargs from GNU find-utils need
> '-i/--replace' to
> > replace {} with the content from stdin. After the change this will be
> equivilent to
> > 'xargs rm -f', i.e. without argument. That wouldn't work.
> >
> My bad. I was confused by the {} were kept. Afaik those could be
> removed as well.

No, you're right. The {} must be removed as well. Committed.

-- Hisham
_______________________________________________
gobolinux-devel mailing list
gobolinux-devel@lists.gobolinux.org
http://lists.gobolinux.org/mailman/listinfo/gobolinux-devel

Reply via email to