[Bro] Strange Bro build problem with yacc/bison

Christian Kreibich 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.

*shrug*

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?

Cheers,
Christian.
-- 
________________________________________________________________________
                                          http://www.cl.cam.ac.uk/~cpk25
                                                    http://www.whoop.org




More information about the Bro mailing list