From: Daniel Phillips <phillips@bonn-fries.net>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Sam Ravnborg <sam@ravnborg.org>,
Thunder from the hill <thunder@ngforever.de>,
Ion Badulescu <ionut@cs.columbia.edu>,
linux-kernel@vger.kernel.org
Subject: Re: KBuild 2.5 Impressions
Date: Sun, 2 Jun 2002 17:16:34 +0200 [thread overview]
Message-ID: <E17EX5v-0000Qd-00@starship> (raw)
In-Reply-To: <Pine.LNX.4.44.0206012349360.671-100000@age.cs.columbia.edu> <E17EWV7-0000Pv-00@starship> <20020602165643.A1940@mars.ravnborg.org>
On Sunday 02 June 2002 16:56, Sam Ravnborg wrote:
> On Sun, Jun 02, 2002 at 04:38:33PM +0200, Daniel Phillips wrote:
> > You mean, fixing the bugs you introduced by trying to add it piecemeal?
> > How about breaking it up where it makes sense to do so, and not breaking
> > it up where it's silly.
>
> Can we agree that it makes sense to add features one-by-one when
> they are independent?
Oh absolutely, and have you looked at the current factoring?
http://marc.theaimsgroup.com/?a=102296100300003&r=1&w=2
This is still being improved, of course.
> If thats the case then it is a simple matter of looking through the
> features already present in kbuild-2.5.
> Then to compare those features with the work done by Kai.
Well, actually a lot of the work done by Kai is simply importing
portions of Keith's work that break out easily, which is purely
duplication of effort, since such work is already in progress. In
fact it creates more work, because then we have to go parse Kai's
patches and find out what he submitted, then see if it gets applied
so we can mark it 'applied' in the list. This is a real waste of
time, and did I mention, it's divisive?
> If the feature is worth it and can be introduced without the core,
> then introduce it in kbuild-2.4.
> This will make this specific feature visible to many people, and
> those who feel against it can speak up.
>
> > If you have a specific suggestion about which part should be broken out,
> > feel free to provide details.
>
> I already gave a list of features that could be broken out. Do you request
> more information than that?
Yes, those seem to be good suggestions. What I'd suggest is: import
enough of kbuild 2.5 to support the feature (in some case nothing
needs to be imported), then make it work also for old kbuild (in
some cases that will require no work. This I'd call cooperation,
which would look good on everybody involved.
I'm not working on this, please pass your specific suggestions to:
Thunder from the hill <thunder@ngforever.de>
> I already submitted 4 patches digged out from kbuild-2.5, one of them
> introducing a bug. The bug was present in kbuild-2.5!
> This bug was easy to spot since the patch was selfcontained, but
> within several thousands of kbuild-2.5 source lines it would have been missed
> most probarly.
Good point. The bright side of all this is, we're getting more eyeballs
than ever actually looking at the code, and chances are, even thinking
about how it works.
By the way, kbuild 2.5 running on 2.4.19-pre9 turns in a build time of
5.4 seconds when nothing needs to be rebuilt and 8.3 seconds after
touch fs/ext2/inode.c, on my 2 x 1 GHz compile box. And so far, hasn't
embarrassed itself once.
--
Daniel
next prev parent reply other threads:[~2002-06-02 15:17 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-05-30 9:45 KBuild 2.5 Impressions Daniel Phillips
2002-05-30 12:50 ` Martin Dalecki
2002-05-30 14:08 ` Kenneth Johansson
2002-05-30 13:28 ` Paul P Komkoff Jr
2002-05-30 21:55 ` Ion Badulescu
2002-05-30 22:29 ` Thunder from the hill
2002-05-30 23:19 ` Daniel Phillips
2002-05-31 0:09 ` David Lang
2002-05-31 0:29 ` Daniel Phillips
2002-05-31 0:32 ` Larry McVoy
2002-06-03 7:21 ` Rusty Russell
2002-05-31 0:13 ` Kenneth Johansson
2002-05-31 0:47 ` Daniel Phillips
2002-05-31 1:20 ` Skip Ford
2002-05-31 1:24 ` Daniel Phillips
2002-05-31 13:48 ` Tomas Szepe
2002-05-31 14:28 ` Nicolas Pitre
2002-05-31 15:21 ` Thunder from the hill
[not found] ` <Pine.LNX.4.44.0205302129120.29405-100000@hawkeye.luckynet.adm>
2002-05-31 4:01 ` Skip Ford
2002-05-31 4:46 ` KBuild 2.5 Migration Daniel Phillips
2002-05-31 5:12 ` Skip Ford
2002-05-31 0:48 ` KBuild 2.5 Impressions Nicolas Pitre
2002-05-31 3:29 ` Hua Zhong
2002-05-31 3:39 ` Daniel Phillips
2002-05-31 3:55 ` Thunder from the hill
2002-06-02 4:03 ` Ion Badulescu
2002-06-02 6:51 ` Daniel Phillips
2002-06-02 7:58 ` Thunder from the hill
2002-06-02 14:03 ` Sam Ravnborg
2002-06-02 14:38 ` Daniel Phillips
2002-06-02 14:56 ` Sam Ravnborg
2002-06-02 15:05 ` Thunder from the hill
2002-06-02 15:16 ` Daniel Phillips [this message]
2002-06-02 18:26 ` Sam Ravnborg
2002-06-02 18:39 ` Thunder from the hill
2002-06-02 18:57 ` Ion Badulescu
2002-06-02 19:15 ` Thunder from the hill
2002-06-03 2:04 ` Kai Germaschewski
2002-06-02 15:47 ` Thunder from the hill
2002-06-02 11:21 ` Peter Osterlund
2002-06-02 11:37 ` Thunder from the hill
2002-06-02 11:53 ` Peter Osterlund
2002-06-02 12:03 ` Thunder from the hill
2002-06-02 12:09 ` Thunder from the hill
2002-06-02 12:51 ` Peter Osterlund
2002-06-02 14:00 ` Daniel Phillips
-- strict thread matches above, loose matches on Subject: below --
2002-06-03 2:49 Dan Kegel
2002-06-03 3:28 ` Linus Torvalds
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=E17EX5v-0000Qd-00@starship \
--to=phillips@bonn-fries.net \
--cc=ionut@cs.columbia.edu \
--cc=linux-kernel@vger.kernel.org \
--cc=sam@ravnborg.org \
--cc=thunder@ngforever.de \
/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