SUPERSEDES APPROVE
On Fri, 2005-07-15 at 23:30 +0200, Karl Janmar wrote:
> Hi,
Hi, and sorry for the delay,
> In the scripts 'grep-all-files-in-current-directory-*' in
> lib/xemacs/xemacs-packages/lisp/xemacs-base/compile.el
>
> There is a test that check if the system has GNU find and GNU xargs.
> Unfortunantly the test is incomplete, it only verfy that find can handle
> the parameter it want to give it, no verification is done on xargs.
>
> I have made a patch that make 'grep-all-files-in-current-directory-*'
> work again on NetBSD and FreeBSD.
>
> patch is attached.
Thanks! After briefly looking into this, I noticed that it'd be good to
test whether xargs groks -0 too (on Solaris 8 it doesn't, even though it
does understand -e). I think the attached implementation would be ok.
Reviewers: I'll commit early next week unless there are objections.
2005-10-23 Ville Skyttä <scop(a)xemacs.org>
* compile.el (grep-find-use-xargs): Test "GNU" compatibility of
xargs too in addition to find, and fall back to using generic
find+xargs if the latter is available. Thanks to Karl Janmar for
the initial implementation.