From owner-freebsd-commit Wed Nov 29 10:49:44 1995 Return-Path: owner-commit Received: (from root@localhost) by freefall.freebsd.org (8.6.12/8.6.6) id KAA09760 for freebsd-commit-outgoing; Wed, 29 Nov 1995 10:49:44 -0800 Received: (from root@localhost) by freefall.freebsd.org (8.6.12/8.6.6) id KAA09705 for cvs-all-outgoing; Wed, 29 Nov 1995 10:48:21 -0800 Received: (from root@localhost) by freefall.freebsd.org (8.6.12/8.6.6) id KAA09692 for cvs-usrsbin-outgoing; Wed, 29 Nov 1995 10:48:17 -0800 Received: from Root.COM (implode.Root.COM [198.145.90.17]) by freefall.freebsd.org (8.6.12/8.6.6) with ESMTP id KAA09685 ; Wed, 29 Nov 1995 10:48:09 -0800 Received: from corbin.Root.COM (corbin [198.145.90.50]) by Root.COM (8.6.12/8.6.5) with ESMTP id KAA01389; Wed, 29 Nov 1995 10:48:06 -0800 Received: from localhost (localhost [127.0.0.1]) by corbin.Root.COM (8.6.12/8.6.5) with SMTP id KAA00293; Wed, 29 Nov 1995 10:46:28 -0800 Message-Id: <199511291846.KAA00293@corbin.Root.COM> To: "Justin T. Gibbs" cc: CVS-committers@freefall.freebsd.org, cvs-usrsbin@freefall.freebsd.org Subject: Re: cvs commit: src/usr.sbin/config Makefile main.c In-reply-to: Your message of "Wed, 29 Nov 95 10:34:40 PST." <199511291834.KAA08699@freefall.freebsd.org> From: David Greenman Reply-To: davidg@Root.COM Date: Wed, 29 Nov 1995 10:46:28 -0800 Sender: owner-commit@FreeBSD.ORG Precedence: bulk >>That's impossible to do given the current scheme of 'everything is an >>option'. However, I think a much better solution to the problem is to >>have config kick out a 'remember to do a make clean' if the directory >>exists. If the user chooses not to do that, then it's their problem not >>ours. > >Why not make option lines add an entry to a common header file. Only >modules that export an option need to include the "option" header. >We could have a separate keyword "define" that gives the current behavior >so you can still do it the old quick way for debugging. I think that >a make depend would have much more value if this was done. The other idea is to add an option file for each supported option. Some people don't like this because it makes adding a new option a little more difficult (all the options would have to be listed in an options file, and you need to add a #include plus a #ifdef to each file that uses the option). The idea is that the option file would have a name like "option_foobar.h" or something deterministic which would be autogenerated by config. The file would either have the #define or it would be null (consuming no disk space and would be relative fast access). This approach allows for fine control over the kernel options dependency. -DG