Paolo Bonzini wrote: > On 11/14/2010 03:09 PM, Jim Meyering wrote: >>> It is possible to disable the body of these two expansions by >>> assigning values to GREP and EGREP inside the configure script. >> >> But with no grep program already installed, that will always fail. > > You can implement a basic grep that is "good enough for configure" > using while/read and case. You will use the newly-built grep > after it's, well, built. But nobody uses grep in "make all", so > it can be substituted unconditionally in Makefiles, and a shell > function can replace the dummygrep binary. > > This is the patch: > > 2010-11-14 Paolo Bonzini <bonz...@gnu.org> > > * configure.ac (fn_grep): New. Set GREP and EGREP to it, > replace with newly-built grep before AC_OUTPUT.
Good idea to use a shell function. That patch looks fine, but please reference the bug report and reporter in the commit log.
