sys/Makefile .if defined(MODULES_WITH_WORLD)

From: Julian H. Stacey <jhs_at_berklix.com>
Date: Mon, 09 Jul 2018 11:50:01 +0200
Hi current_at_
src/sys/dev/amdsbwd/amdsbwd.c broke src/sys/modules

Is it immediately intuitive & well known to developers working in sys/dev
to enable MODULES_WITH_WORLD before a test make all before a commit ?

Or what should we do to increase the liklehood of commiters catching
modules/ errors before a commit ?

With src/
	.ctm_status src-cur 13573
	.svn_revision 335362
sys/Makefile has
	.if defined(MODULES_WITH_WORLD)
	SUBDIR+=modules
& nothing from cd /usr/src; find . -name \*src.conf\*
& no default /etc/src.conf with no
	MODULES_WITH_WORLD=YES
so make all does not build /sys/modules/ 
so this not seen from /sys/modules/
===> amdsbwd (all)
cc  -O2 -pipe -DBERKLIX=YES  -fno-strict-aliasing -Werror -D_KERNEL -DKLD_MODULE -nostdinc   -I. -I/data/release/s1/usr/src/sys -I/data/release/s1/usr/src/sys/contrib/ck/include -fno-common  -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer   -MD  -MF.depend.amdsbwd.o -MTamdsbwd.o -mcmodel=kernel -mno-red-zone -mno-mmx -mno-sse -msoft-float  -fno-asynchronous-unwind-tables -ffreestanding -fwrapv -fstack-protector -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Wcast-qual -Wundef -Wno-pointer-sign -D__printf__=__freebsd_kprintf__ -Wmissing-include-dirs -fdiagnostics-show-option -Wno-unknown-pragmas -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -Wno-error-unused-function -Wno-error-pointer-sign -Wno-error-shift-negative-value -Wno-address-of-packed-member  -mno-aes -mno-avx  -std=iso9899:1999 -c /data/release/s1/usr/src/sys/dev/amdsbwd/amdsbwd.c -o amdsbwd.o
/data/release/s1/usr/src/sys/dev/amdsbwd/amdsbwd.c:52:10: fatal error: 
      'opt_amdsbwd.h' file not found
#include "opt_amdsbwd.h"


PS With 
	.ctm_status	src-cur 13601
	.svn_revision	336117
nothing from
	find . -name opt_amdsbwd.h
but this has
	src/sys/dev/amdsbwd/amdsbwd.c
	#include "opt_amdsbwd.h"
I haven't yet upgraded my src/ yet to see if it still fails.

Cheers,
Julian
-- 
Julian Stacey, Computer Consultant, Systems Engineer, BSD Linux Unix, Munich
 Brexit Referendum stole 3.7 million votes inc. from 700,000 British in EU.
 UK Goverment lies it's democratic in Article 50 paragraph 3 of letter to EU.
 http://berklix.eu/queen/  https://www.peoples-vote.uk   193,000 _at_ 8 Jul 2018
Received on Mon Jul 09 2018 - 08:55:41 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:17 UTC