Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Jim Meyering
Ralf Wildenhues <[EMAIL PROTECTED]> wrote: > * Jim Meyering wrote on Sun, Oct 05, 2008 at 07:42:45PM CEST: >> Bruno Haible <[EMAIL PROTECTED]> wrote: >> > >> > \> is also not a portable piece of regular expression: not in a basic >> > regex, >> > and not in an extended regex. It is a GNU extensio

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Ralf Wildenhues
* Jim Meyering wrote on Sun, Oct 05, 2008 at 07:42:45PM CEST: > Bruno Haible <[EMAIL PROTECTED]> wrote: > > > > \> is also not a portable piece of regular expression: not in a basic regex, > > and not in an extended regex. It is a GNU extension. > > It works with Solaris 10's grep, > so maybe it w

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Jim Meyering
Bruno Haible <[EMAIL PROTECTED]> wrote: > Jim Meyering wrote: >> Unfortunate, but true. >> Thanks. How about this? >> ... >> +grep '^[ ]*A[CM]_PROG_LIBTOOL\>' configure.ac >/dev/null \ > > \> is also not a portable piece of regular expression: not in a basic regex, > and not in an exte

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Bruno Haible
Jim Meyering wrote: > Unfortunate, but true. > Thanks. How about this? > ... > +grep '^[ ]*A[CM]_PROG_LIBTOOL\>' configure.ac >/dev/null \ \> is also not a portable piece of regular expression: not in a basic regex, and not in an extended regex. It is a GNU extension. Bruno

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Jim Meyering
Bruno Haible <[EMAIL PROTECTED]> wrote: >> -grep '^[ ]*A[CM]_PROG_LIBTOOL\>' configure.ac >/dev/null || >> +grep -E '^[ ]*(LT_INIT|A[CM]_PROG_LIBTOOL)\>' configure.ac >> >/dev/null || > > Solaris 9 'grep' does not support the '-E' option. This is documented in the > autoconf m

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Bruno Haible
Hi Jim, > -grep '^[ ]*A[CM]_PROG_LIBTOOL\>' configure.ac >/dev/null || > +grep -E '^[ ]*(LT_INIT|A[CM]_PROG_LIBTOOL)\>' configure.ac > >/dev/null || Solaris 9 'grep' does not support the '-E' option. This is documented in the autoconf manual: Traditional `grep' implementatio

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Jim Meyering
Ralf Wildenhues <[EMAIL PROTECTED]> wrote: > * Jim Meyering wrote on Sun, Oct 05, 2008 at 01:01:14PM CEST: >> "Debarshi Ray" <[EMAIL PROTECTED]> wrote: >> >> Subject: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as >> >> A

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Ralf Wildenhues
* Jim Meyering wrote on Sun, Oct 05, 2008 at 01:01:14PM CEST: > "Debarshi Ray" <[EMAIL PROTECTED]> wrote: > >> Subject: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as > >> AM_PROG_LIBTOOL > >> * build-aux/bootstrap: Check for AC_PROG_LIBTOOL

Re: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as AM_PROG_LIBTOOL

2008-10-05 Thread Jim Meyering
"Debarshi Ray" <[EMAIL PROTECTED]> wrote: >> Subject: [PATCH] bootstrap: check for AC_PROG_LIBTOOL as well as >> AM_PROG_LIBTOOL > > Wow! Thanks a lot. Will this make its way into build-aux/bootstrap of > Gnulib as well? That *was* for gnulib. I'm Cc