fe815331bb
The current default is provided in various Makefile.inc in some top-level directories and covers a good portion of the tree, but doesn't cover parts of the build a little deeper (e.g. libcasper). Provide a default in src.sys.mk and set WARNS to it in bsd.sys.mk if that variable is defined. This lets us relatively cleanly provide a default WARNS no matter where you're building in the src tree without breaking things outside of the tree. Crunchgen has been updated as a bootstrap tool to work on this change because it needs r365605 at a minimum to succeed. The cleanup necessary to successfully walk over this change on WITHOUT_CLEAN builds has been added. There is a supplemental project to this to list all of the warnings that are encountered when the environment has WARNS=6 NO_WERROR=yes: https://warns.kevans.dev -- this project will hopefully eventually go away in favor of CI doing a much better job than it. Reviewed by: emaste, brooks, ngie (all earlier version) Reviewed by: emaste, arichardson (depend-cleanup.sh change) Differential Revision: https://reviews.freebsd.org/D26455
48 lines
1.6 KiB
Makefile
48 lines
1.6 KiB
Makefile
# $FreeBSD$
|
|
|
|
# Note: This file is also duplicated in the sys/conf/kern.pre.mk so
|
|
# it will always grab SRCCONF, even if it isn't being built in-tree
|
|
# to preserve historical (and useful) behavior. Changes here need to
|
|
# be reflected there so SRCCONF isn't included multiple times.
|
|
|
|
.if !defined(_WITHOUT_SRCCONF)
|
|
# Allow user to configure things that only effect src tree builds.
|
|
SRCCONF?= /etc/src.conf
|
|
.if !empty(SRCCONF) && \
|
|
(exists(${SRCCONF}) || ${SRCCONF} != "/etc/src.conf") && \
|
|
!target(_srcconf_included_)
|
|
|
|
# Validate that the user didn't try setting an env-only variable in
|
|
# their src.conf. This benefits from already including bsd.mkopt.mk.
|
|
.for var in ${__ENV_ONLY_OPTIONS:O:u}
|
|
__presrcconf_${var}:= ${MK_${var}:U-}${WITHOUT_${var}:Uno:Dyes}${WITH_${var}:Uno:Dyes}
|
|
.endfor
|
|
|
|
.sinclude "${SRCCONF}"
|
|
_srcconf_included_: .NOTMAIN
|
|
|
|
# Validate the env-only variables.
|
|
.for var in ${__ENV_ONLY_OPTIONS:O:u}
|
|
__postrcconf_${var}:= ${MK_${var}:U-}${WITHOUT_${var}:Uno:Dyes}${WITH_${var}:Uno:Dyes}
|
|
.if ${__presrcconf_${var}} != ${__postrcconf_${var}}
|
|
.error Option ${var} may only be defined in ${SRC_ENV_CONF}, environment, or make argument, not ${SRCCONF}.
|
|
.endif
|
|
.undef __presrcconf_${var}
|
|
.undef __postrcconf_${var}
|
|
.endfor
|
|
|
|
.endif # SRCCONF
|
|
.endif
|
|
|
|
# The following should be removed no earlier than LLVM11 being imported into the
|
|
# tree, to ensure we don't regress the build. LLVM11 and GCC10 will switch the
|
|
# default over to -fno-common, making this redundant.
|
|
CFCOMMONFLAG?= -fno-common
|
|
CFLAGS+= ${CFCOMMONFLAG}
|
|
|
|
DEFAULTWARNS= 6
|
|
|
|
# tempting, but bsd.compiler.mk causes problems this early
|
|
# probably need to remove dependence on bsd.own.mk
|
|
#.include "src.opts.mk"
|