Re: bison-2.6.5 1 test failed on solaris 10

[Dennis Clarke]

merci beaucoup !

dc

----- Original Message -----
From: Akim Demaille <address@hidden>
Date: Thursday, November 8, 2012 1:42 pm
Subject: Re: bison-2.6.5 1 test failed on solaris 10
To: Dennis Clarke <address@hidden>
Cc: address@hidden


> Le 8 nov. 2012 à 17:34, Akim Demaille a écrit :
> 
> > This is minor: grep -E does not seem to support -w.  I'll adjust the 
> test suite, thanks a lot for the report.
> 
> I installed this (in branch-2.6).
> 
> commit 3a526f5cc3b45fc68d06de0b5332105113741093
> Author: Akim Demaille <address@hidden>
> Date:   Thu Nov 8 19:34:29 2012 +0100
> 
>     tests: don't expect $EGREP to support -w
>     
>     Does not work on Solaris 10.  Reported by Dennis Clarke.
>     http://lists.gnu.org/archive/html/bug-bison/2012-11/msg00009.html
>     * tests/headers.at (Several parsers): Use Perl instead.
>     While at it, run it only once, on all the generated headers.
>     Adjust to YY_NULL be defined in position.hh.
> 
> diff --git a/THANKS b/THANKS
> index 77d35be..4e88e8f 100644
> --- a/THANKS
> +++ b/THANKS
> @@ -31,6 +31,7 @@ Dagobert Michelsen        address@hidden
>  Daniel Hagerty            address@hidden
>  David J. MacKenzie        address@hidden
>  David Kastrup             address@hidden
> +Dennis Clarke             address@hidden
>  Derek M. Jones            address@hidden
>  Di-an Jan                 address@hidden
>  Dick Streefland           address@hidden
> diff --git a/tests/headers.at b/tests/headers.at
> index a60e315..7df4cc9 100644
> --- a/tests/headers.at
> +++ b/tests/headers.at
> @@ -163,21 +163,6 @@ exp:
>  
>  AT_BISON_CHECK([-d -o AT_SKEL_CC_IF([$1.cc $1.yy], [$1.c $1.y])])
>  
> -# Check there is no 'yy' left.
> -# C++ output relies on namespaces and still uses yy a lot.
> -AT_SKEL_CC_IF([],
> -  [AT_CHECK([$EGREP yy $1.h], [1])])
> -
> -# Check there is not 'YY' left.
> -# Ignore comments, YYPARSE_PARAM (obsolete), YYPUSH_MORE(_DEFINED)?
> -# (constant definition), YY_\w+_INCLUDED (header guards).
> -#
> -# YYDEBUG (not renamed) can be read, but not changed.
> -AT_CHECK([[sed -ne 's,/\*[^*]*\*/,,g;s,//.*,,' \
> -                -e '/YY/p' ]$1.AT_SKEL_CC_IF([hh], [h])[ |
> -     $EGREP -wv 
> 'YY(PARSE_PARAM|PUSH_MORE(_DEFINED)?|_[0-9A-Z_]+_INCLUDED)|(defined|if) 
> YYDEBUG']],
> -         [1])
> -
>  AT_LANG_COMPILE([$1.o])
>  AT_CHECK([[echo "$1" >>expout]])
>  
> @@ -240,6 +225,36 @@ AT_TEST([x7], [%define api.push-pull both])
>  AT_TEST([x8], [%define api.pure %define api.push-pull both])
>  #AT_TEST([x5], [%locations %language "c++" %glr-parser])
>  
> +# Check there is no 'yy' left.
> +# C++ output relies on namespaces and still uses yy a lot.
> +#
> +# Check there is no 'YY' left.
> +# Ignore comments, YYPARSE_PARAM (obsolete), YYPUSH_MORE(_DEFINED)?
> +# (constant definition), YY_\w+_INCLUDED (header guards).
> +#
> +# YYDEBUG (not renamed) can be read, but not changed.
> +AT_CHECK([[$PERL -n -0777 -e '
> +  s{/\*.*?\*/}{}gs;
> +  s{//.*}{}g;
> +  s{\b(YYPARSE_PARAM
> +      |YYPUSH_MORE(_DEFINED)?
> +      |YY_\w+_INCLUDED
> +      |YY_NULL
> +      |(defined|if)\ YYDEBUG
> +      )\b}{}gx;
> +  while (/^(.*YY.*)$/gm)
> +  {
> +    print "$ARGV: $1\n";
> +  }
> +  if ($ARGV =~ /\.h$/)
> +  {
> +    while (/^(.*yy.*)$/gm)
> +    {
> +      print "$ARGV: $1\n";
> +    }
> +  }
> +' -- *.hh *.h]])
> +
>  # Check that the headers are self-contained, and protected against
>  # multiple inclusions.  While at it, check they are sane for C++.
>  for h in *.h *.hh
>