Re: [PATCH] kconfig: menuconfig and config uses $objtree

Keith Owens (kaos@ocs.com.au)
Sat, 29 Jun 2002 11:50:41 +1000


On Fri, 28 Jun 2002 19:28:07 +0200,
Sam Ravnborg <sam@ravnborg.org> wrote:
>It basically teach the config tools that the SRC are no longer
>in current directory but pointed out by $srctree, that output files
>are pointed out by $objtree, and temporary files the same place.
>
>No nasty tricks with sym-lnks required, no copy files around before
>or after the config tools are used. No specific directories that
>needs to be created beforehand.
>Indeed my approach is a number of lines - but that on the other hand
>simplify the usage of the config tools.
>
>But I see your point that we should avoid hardcoding too much
>knowledge in the config tools, and I may change the patch to
>use command-line parameters to specify SRC and OBJ dirs.

What happens when you want to support multiple source trees? Your
approach prevents the use of multiple source trees, either shadow or
separate (add on) code and breaks kbuild 2.5. My approach puts all the
complexity (once) in the makefile, not replicated over multiple CML
programs.

What happens when the config data is not in monolithic files but is
supplied in per-driver files (driver.inf)? Linus wants that feature
eventually. Note that driver.inf will contain more than just config
data, it will contain all the data required to build a driver. With
your approach every CML program would have to be changed to understand
the format of the driver.inf files, replicating the code over multiple
parsers. With my approach you need one program that extracts the
relevant data for config and builds the config tree, then the existing
CML programs run unchanged.

Do it once in the makefile. Do not embed the knowledge about the
source of the config data in every CML program.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/