[Bro] Strange Bro build problem with yacc/bison
christian at whoop.org
Mon Dec 4 14:32:19 PST 2006
On Mon, 2006-12-04 at 14:13 -0700, frenzy at frenzy.org wrote:
> This works (without the patch). :) I forgot to mention that we had run
> through that as well. I wasn't sure if the parse.y needed to be included
> or not. Is there going to be a problem on systems that don't have
> yacc installed? Or was it required anyway?
I believe we currently ship generated yacc output. I had forgotted about
that -- the patch I posted actually conflicts with that approach because
it removes files that could only be regenerated with yacc/bison.
I'm not sure what people prefer. Dropping the generated files from the
tarball is the cleanest solution, but we then require (f)lex/
yacc/bison. If we don't drop them, we could force invocation of
make_parser.pl by tweaking the Makefile rules.
Btw, make_parser.pl is only ever used in the parse.y case, but we also
have re-parse.y and rule-parse.y. Do the "@"-issues you're seeing not
occur with the latter two?
More information about the Bro