mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Keith Owens <kaos@ocs.com.au>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Kai Germaschewski <kai@tp1.ruhr-uni-bochum.de>,
	mec@shout.net, kbuild-devel@lists.sourceforge.net,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] kconfig: menuconfig and config uses $objtree
Date: Sat, 29 Jun 2002 11:50:41 +1000	[thread overview]
Message-ID: <5050.1025315441@ocs3.intra.ocs.com.au> (raw)
In-Reply-To: Your message of "Fri, 28 Jun 2002 19:28:07 +0200." <20020628192807.A2142@mars.ravnborg.org>

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.


  reply	other threads:[~2002-06-29  1:48 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-27 22:14 Sam Ravnborg
2002-06-28  2:19 ` Keith Owens
2002-06-28 17:28   ` Sam Ravnborg
2002-06-29  1:50     ` Keith Owens [this message]
2002-06-29  7:26       ` Sam Ravnborg
2002-06-29  8:23         ` Keith Owens
2002-06-30  9:31         ` [kbuild-devel] " Greg Banks
2002-06-29 15:36       ` Roman Zippel
2002-06-28  8:07 ` [kbuild-devel] " Greg Banks
2002-06-28  8:58   ` Keith Owens
2002-06-28  9:16     ` Greg Banks

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5050.1025315441@ocs3.intra.ocs.com.au \
    --to=kaos@ocs.com.au \
    --cc=kai@tp1.ruhr-uni-bochum.de \
    --cc=kbuild-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mec@shout.net \
    --cc=sam@ravnborg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®