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.