svn commit: r230972 - in head: share/mk sys/amd64/conf sys/i386/conf sys/modules sys/modules/drm sys/modules/sound/driver sys/modules/usb tools/build/options

Robert Millan rmh at freebsd.org
Fri Feb 10 20:09:08 UTC 2012


Hi all,

Sorry for the late reply.  I can't keep up with mail during the week.

2012/2/4, Bjoern A. Zeeb <bzeeb-lists at lists.zabbadoz.net>:
> And can you please include GENERIC at least in these kernel configs?  They
> are not building and this breaks a make universe and I guess eventually the
> tinderbox as well...

Including GENERIC sounds problematic because if both WITHOUT_* files
are included, then GENERIC is included twice!  As discussed on -arch,
the idea was that users would never build WITHOUT_* directly, but
rather include them in their custom kernel config.

These files weren't intended to be picked up by "make universe" either
[1]. It just didn't occur to me that the universe target would pick
them up.

Here's a possible solution to this patch (untested).  In case it looks
fine to everyone (and my mentors approve it), I'll definitely test
with "make universe" before commit.

[1] shame on me for not testing lasest version of my patch with "make
universe" -- I only tested the first one which didn't add WITHOUT_*
files.

-- 
Robert Millan
-------------- next part --------------
Index: Makefile
===================================================================
--- Makefile	(revision 230971)
+++ Makefile	(working copy)
@@ -404,7 +404,7 @@
 .endif
 KERNCONFS!=	cd ${KERNSRCDIR}/${TARGET}/conf && \
 		find [A-Z0-9]*[A-Z0-9] -type f -maxdepth 0 \
-		! -name DEFAULTS ! -name NOTES
+		! -name DEFAULTS ! -name NOTES ! -name WITHOUT_\*
 universe_kernconfs:
 .for kernel in ${KERNCONFS}
 TARGET_ARCH_${kernel}!=	cd ${KERNSRCDIR}/${TARGET}/conf && \


More information about the svn-src-all mailing list