Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 Apr 2002 17:05:59 -0400 (EDT)
From:      John Baldwin <jhb@FreeBSD.org>
To:        Ruslan Ermilov <ru@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, re@FreeBSD.org
Subject:   Re: cvs commit: src Makefile Makefile.inc1 src/etc Makefile src/
Message-ID:  <XFMail.20020426170559.jhb@FreeBSD.org>
In-Reply-To: <20020426203600.GA69757@sunbay.com>

next in thread | previous in thread | raw e-mail | index | archive | help

On 26-Apr-2002 Ruslan Ermilov wrote:
> On Fri, Apr 26, 2002 at 04:17:27PM -0400, John Baldwin wrote:
>> 
>> On 26-Apr-2002 Ruslan Ermilov wrote:
>> >   Log:
>> >   Milestone #1 in cross-arch make releases.
>> 
>> I'm sure re@ or qa@ would have loved to have had a chance to review this
>> before
>> it went in.
>> 
> Huh?  My initial message subjected "Cross-platform releases" was CC:'ed
> to re@FreeBSD as well.  None from re@ replied to my message.  Is it
> probably a good time for me to apply to join re@?  :-)

The original message didn't include a patch. :)  The idea is certainly
something we would like to see, but the details are something your message
did not talk about.

>> >   In release.3 and doMFSKERN, build kernels in the "world"
>> >   environment.  KERNELS now means "additional" kernels, GENERIC is
>> >   always built.
>> 
>> This is wrong.  Not everyone wants to use GENERIC.  Instead, please use
>> the approach of a patch green has worked up that replaces KERNELS with two
>> variables:
>> 
>> DEFAULTKERNEL?= GENERIC
>> #EXTRAKERNELS?=
>> 
>> Where DEFAULTKERNEL is always built and is installed as /boot/kernel/kernel
>> on CD's, and in the base dist, etc.  EXTRAKERNELS is an optional list
>> similar
>> to what you have done with KERNELS.  We should not specifically tie people
>> to
>> using GENERIC as the default kernel.  For people who build custom releases,
>> it
>> should be possible to use a different kernel config besides GENERIC for the
>> default kernel install, yet still include a GENERIC kernel in the release as
>> a
>> fall-back kernel.
>> 
> Probably, but my patch did not make things worse.  Old version (before my
> patch) assumed that GENERIC was always present in KERNELS, and used it as
> the _main_ kernel:

I know, and green has already developed and tested a patch as I mentioned above
which I would have encouraged you to incorporiate if you had asked for review. 

>> >   Inline createBOOTMFS target.
>> 
>> Why?
>>  
> Because it's now used only once.  I think that calling it in doMFSKERN
> in the old version was an oversight too.

Well, this should likely have been a separate commit from adding cross-release
support as it's not very related.

>> >   Use already built GENERIC kernel modules to augment mfsfd's
>> >   /stand/modules.  GC doMODULES as such.
>> 
>> This assumes too much about GENERIC, IMO.  Eventually we might use a
>> separate
>> kernel config that just builds modules and no actual kernel for this type of
>> stuff.
>> 
> The only assumption made is that _all_ modules are built with GENERIC.
> This is always true unless MODULES_OVERRIDE is set (which it apparently
> is not).  Moreover, BOOTMFS (for which we create modules) is by design
> a reduced version of GENERIC.

In the future we will have something much more general than MODULES_OVERRIDE
and you just wiped out the abstraction in the release makefile that would let
us more smoothly handle that when it comes.

-- 

John Baldwin <jhb@FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve!"  -  http://www.FreeBSD.org/

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.20020426170559.jhb>