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>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2012-05-22 12:22, Dag-Erling Sm=C3=B8rgrav wrote:
> Baptiste Daroussin <bapt@FreeBSD.org> writes:
>> Log:
>>   Import byacc from invisible island, it brings us lots of compatibili=
ties with
>>   bison, keeping full compatibility with our previous yacc
>>   implementation.
>=20
> 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...




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