Disable K&R style function definitions for WARNS=6.

Unfortunately there are two slight problems with that:

- Yacc and lex might generate code that generates warnings because of
  this. Require yacc and lex to be rebuilt during bootstrap. I'm not
  incrementing __FreeBSD_version here, because I assume someone else
  will do this eventually.

- When running `make buildkernel', it uses share/mk from the source
  treeo to build aicasm. Because aicasm also depends on lex, this would
  break. Lower WARNS to 5 for now. We should just increment it to 6
  again somewhere in the very far future.
This commit is contained in:
Ed Schouten 2009-12-31 00:07:26 +00:00
parent 3dbee1749a
commit 03bc68ca09
3 changed files with 10 additions and 2 deletions

View File

@ -940,6 +940,11 @@ _mklocale= usr.bin/mklocale
_sed= usr.bin/sed _sed= usr.bin/sed
.endif .endif
.if ${BOOTSTRAPPING} < 900006
_lex= usr.bin/lex
_yacc= usr.bin/yacc
.endif
.if ${BOOTSTRAPPING} < 700018 .if ${BOOTSTRAPPING} < 700018
_gensnmptree= usr.sbin/bsnmpd/gensnmptree _gensnmptree= usr.sbin/bsnmpd/gensnmptree
.endif .endif
@ -966,6 +971,8 @@ bootstrap-tools:
${_mklocale} \ ${_mklocale} \
usr.bin/rpcgen \ usr.bin/rpcgen \
${_sed} \ ${_sed} \
${_lex} \
${_yacc} \
usr.bin/xinstall \ usr.bin/xinstall \
${_gensnmptree} \ ${_gensnmptree} \
usr.sbin/config \ usr.sbin/config \

View File

@ -47,7 +47,8 @@ CWARNFLAGS += -Wreturn-type -Wcast-qual -Wwrite-strings -Wswitch\
. endif . endif
# BDECFLAGS # BDECFLAGS
. if ${WARNS} >= 6 . if ${WARNS} >= 6
CWARNFLAGS += -Wchar-subscripts -Winline -Wnested-externs -Wredundant-decls CWARNFLAGS += -Wchar-subscripts -Winline -Wnested-externs\
-Wredundant-decls -Wold-style-definition
. endif . endif
. if ${WARNS} >= 2 && ${WARNS} <= 4 . if ${WARNS} >= 2 && ${WARNS} <= 4
# XXX Delete -Wuninitialized by default for now -- the compiler doesn't # XXX Delete -Wuninitialized by default for now -- the compiler doesn't

View File

@ -15,7 +15,7 @@ SRCS= ${GENHDRS} ${CSRCS} ${YSRCS} ${LSRCS}
CLEANFILES+= ${GENHDRS} ${YSRCS:R:C/(.*)/\1.output/g} CLEANFILES+= ${GENHDRS} ${YSRCS:R:C/(.*)/\1.output/g}
DPADD= ${LIBL} DPADD= ${LIBL}
LDADD= -ll LDADD= -ll
WARNS?= 6 WARNS?= 5
# Correct path for kernel builds # Correct path for kernel builds
# Don't rely on the kernel's .depend file # Don't rely on the kernel's .depend file