This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

[RFA:] Override AM_PROG_LEX in binutils/acinclude.m4


Here's a patch following Axexandre Oliva's hint in 
<URL:http://sources.redhat.com/ml/binutils/2000-12/msg00289.html> (duh,
didn't think of that...) for solving the buggy lex test in the generated
configure.  I briefly considered putting it in the included
bfd/acinclude.m4, but thought that would be wrong: there's currently no
need for lex in bfd and this is a (hopefully) just temporary fix until we
use a newer (released?) automake.

By the way, is the recommended automake snapshot
<URL:ftp://sources.redhat.com/pub/binutils/automake-000227.tar.bz2>
and ditto autoconf used by all developers or is it perhaps obsoleted and
the README-maintainer-mode instructions need replacing?  I get these
kinds of errors from autoconf with every update that causes aclocal to
run, and have to fix "spurious" changes in aclocal.m4 manually:

configure.in:7: AC_TRY_COMPILE was called before AC_ISC_POSIX
configure.in:7: AC_TRY_RUN was called before AC_ISC_POSIX
autoconf: Undefined macros:
***BUG in Autoconf--please report*** AC_FD_MSG
***BUG in Autoconf--please report*** AC_FD_CC
... (etc.)

Ok to commit?

	* acinclude.m4 (AM_PROG_LEX): Override installed definition.
	* aclocal.m4: Regenerate.
	* configure: Regenerate.

Index: acinclude.m4
===================================================================
RCS file: /cvs/src/src/binutils/acinclude.m4,v
retrieving revision 1.2
diff -p -c -r1.2 acinclude.m4
*** acinclude.m4	2000/08/31 09:41:33	1.2
--- acinclude.m4	2001/01/23 17:48:18
*************** ifelse(yes,no,[
*** 14,16 ****
--- 14,32 ----
  AC_DEFUN([CY_WITH_NLS],)
  AC_SUBST(INTLLIBS)
  ])
+ 
+ ## Replacement for AC_PROG_LEX and AC_DECL_YYTEXT
+ ## by Alexandre Oliva <oliva@dcc.unicamp.br>
+ 
+ ## We need to override the installed aclocal/lex.m4 because of a bug in
+ ## this definition in the recommended automake snapshot of 000227:
+ ## There were double-quotes around ``$missing_dir/missing flex'' which was
+ ## bad since aclocal wraps it in double-quotes.
+ 
+ dnl AM_PROG_LEX
+ dnl Look for flex, lex or missing, then run AC_PROG_LEX and AC_DECL_YYTEXT
+ AC_DEFUN(AM_PROG_LEX,
+ [missing_dir=ifelse([$1],,`cd $ac_aux_dir && pwd`,$1)
+ AC_CHECK_PROGS(LEX, flex lex, [$missing_dir/missing flex])
+ AC_PROG_LEX
+ AC_DECL_YYTEXT])

brgds, H-P


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]