Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 22 May 2012 15:32:01 +0200
From:      Dimitry Andric <dim@FreeBSD.org>
To:        =?UTF-8?B?RGFnLUVybGluZyBTbcO4cmdyYXY=?= <des@des.no>
Cc:        svn-src-head@freebsd.org, Baptiste Daroussin <bapt@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org
Subject:   Re: svn commit: r235723 - in head: contrib/byacc tools/regression/usr.bin/yacc usr.bin/yacc usr.bin/yacc/test
Message-ID:  <4FBB9551.6060407@FreeBSD.org>
In-Reply-To: <86likkpla9.fsf@ds4.des.no>
References:  <201205211331.q4LDVRvB060077@svn.freebsd.org> <86likkpla9.fsf@ds4.des.no>

index | next in thread | previous in thread | raw e-mail

On 2012-05-22 12:22, Dag-Erling Smørgrav wrote:
> Baptiste Daroussin <bapt@FreeBSD.org> writes:
>> Log:
>>   Import byacc from invisible island, it brings us lots of compatibilities with
>>   bison, keeping full compatibility with our previous yacc
>>   implementation.
> 
> This commit broke the build, in large part because Baptiste tested with
> Clang instead of GCC, and GCC generates a warning when compiling the
> generated code.

It doesn't seem to compile with clang either, at least not as of r235777:

/usr/src/bin/expr/expr.y:291:2: error: implicit declaration of function 'yyparse' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
        yyparse();
        ^

Note that it looks like the expr.c file is still generated with the
system yacc, so I'm not entirely sure what's going on here...



home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4FBB9551.6060407>