On Thu, Jan 16, 2014 at 07:15:20PM +0100, Pablo Neira Ayuso wrote: > On Thu, Jan 16, 2014 at 06:04:27PM +0000, Patrick McHardy wrote: > > This the recursive parser.h inclusion that happens occasionally when > > regenerating the parser files. Amazingly bison, despite its age, is > > not using header sandwiches to protect against this *sigh* > > > > Slightly ugly, but fixes the problem. > > Ah, much nicer than the "nested too deep" warning message :) Indeed. Pushed out to master. > > > diff --git a/Makefile.rules.in b/Makefile.rules.in > > index 25988dd..ae563a5 100644 > > --- a/Makefile.rules.in > > +++ b/Makefile.rules.in > > @@ -22,7 +22,14 @@ configure: configure.ac > > > > %.c %.h: %.y $(makedeps) > > @echo -e " YACC\t\t$<" > > - $(YACC) $(YACCFLAGS) -d -o $@ $< > > + $(YACC) $(YACCFLAGS) --defines=$*.h.tmp -o $@ $< > > + ( \ > > + echo "#ifndef __$(*F)_H"; \ > > + echo "#define __$(*F)_H"; \ > > + cat $*.h.tmp; \ > > + echo "#endif /* __$(*F)_H */" \ > > + ) > $*.h > > + $(RM) $*.h.tmp > > > > %.c %.h: %.l $(makedeps) > > @echo -e " LEX\t\t$<" > -- > To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html