Re: 07-use-w.patch

[Akim Demaille]

Tom Tromey <address@hidden> writes:

> >>>>> "Akim" == Akim Demaille <address@hidden> writes:
> 
> Akim> As a first step, I can apply this patch *but* with this part and
> Akim> the --Werror part in defs.
> 
> I don't see a `defs' change here.

Hm, I suppose I'm still using `but' incorrectly: I meant that
precisely I don't apply the --Werror part in defs.

> Akim>         * tests/lex2.test, tests/sinclude.test, tests/suffix3.test:
> Akim>         Run automake with --Wno-error.
> 
> Ok.
> 
> Why do we sometimes need --Wno-error?

src/am/tests % cat lex2.test                                     nostromo 13:03
#! /bin/sh

# Test to make sure AC_PROG_LEX is an error.

. $srcdir/defs || exit 1

cat >> configure.in << 'END'
AC_PROG_CC
AC_PROG_LEX
AC_DECL_YYTEXT
END

cat > Makefile.am << 'END'
bin_PROGRAMS = zot
zot_SOURCES = joe.l
LDADD = @LEXLIB@
END

: > joe.l

$AUTOMAKE 2> output || exit 1
test -n "`cat output`"


am/tests/testSubDir % am --foreign                               nostromo 13:03
configure.in: 10: automake requires `AM_PROG_LEX', not `AC_PROG_LEX'
configure.in: 11: `AC_DECL_YYTEXT' is covered by `AM_PROG_LEX'



These are tests were precisely you check that warnings don't exit 1.