Hi Stefano, Am 22.11.2012 um 10:39 schrieb Stefano Lattarini <stefano.lattar...@gmail.com>:
> [+cc bug#12836, automake-patches] > > On 11/21/2012 10:17 PM, Dagobert Michelsen wrote: >> Hi Stefano, >> > Hi Dagobert, thanks for the quick reply. > >> Am 21.11.2012 um 11:11 schrieb Stefano Lattarini >> <stefano.lattar...@gmail.com>: >>> Reference: >>> <http://debbugs.gnu.org/cgi/bugreport.cgi?bug=11524> >>> >>> Any news on this bug? Can it still be reproduced with Automake 1.12.5? >>> Can it be reproduced with the development version of Automake from the >>> 'master' branch? Since the evidence so far seems to point to spurious >>> errors, I will close this report in a few days if I haven't heard back >>> by then. >> >> >> I still get quite some failures: >> http://buildfarm.opencsw.org/~dam/automake-1.12.5-test-suite.log >> > Let's try to go through them one by one (in different replies). > >> FAIL: t/lex-header >> lex-header: running flex --version >> flex version 2.5.4 >> > OK, this version of flex is ~ 15 years old, and ... > >> /bin/bash ./ylwrap `test -f 'lexer.l' || echo './'`lexer.l \ >> lex.yy.c foo-lexer.c -- flex --header-file=mylex.h >> flex: unknown flag '-'. For usage, try >> flex --help >> *** Error code 1 >> make: Fatal error: Command failed for target `foo-lexer.c' >> > ... doesn't support the '--header-file' option. This issue has been > reported once already (in automake bug#12836, which I am CC:ing). > The patch should take care of the issue. Can you confirm it works? Looks good, no more "unknown flag" in test-suite.log Best regards -- Dago > ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- 8< ---- > > From 656f866264c7afe0f821eb0962aad8ff2f65c531 Mon Sep 17 00:00:00 2001 > Message-Id: > <656f866264c7afe0f821eb0962aad8ff2f65c531.1353576830.git.stefano.lattar...@gmail.com> > From: Stefano Lattarini <stefano.lattar...@gmail.com> > Date: Thu, 22 Nov 2012 10:29:37 +0100 > Subject: [PATCH] tests: fix a spurious failure with older flex versions > > See automake bug#11524 and bug#12836. > > * t/lex-header.sh: Here, we use flex option '--header-file', > but that option is not supported by some older flex versions. > Simply skip this test if such an old flex version is detected. > > Signed-off-by: Stefano Lattarini <stefano.lattar...@gmail.com> > --- > t/lex-header.sh | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/t/lex-header.sh b/t/lex-header.sh > index f8d7333..ad2976e 100755 > --- a/t/lex-header.sh > +++ b/t/lex-header.sh > @@ -20,6 +20,12 @@ > required='cc flex' > . ./defs || exit 1 > > +# Here, we need to use the use flex option '--header-file', but some > +# older flex versions don't support is (see automake bug#11524 and > +# bug#12836). Skip this test if such an old flex version is detected. > +$LEX --help | grep '.*--header-file' \ > + || skip_ "flex doesn't support the --header-file' option" > + > cat >> configure.ac << 'END' > AC_PROG_CC > AC_PROG_LEX > -- > 1.8.0.209.gf3828dc -- "You don't become great by trying to be great, you become great by wanting to do something, and then doing it so hard that you become great in the process." - xkcd #896
