X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7E9DF.45C8EFD9@onstor-exch02.onstor.net>; Tue, 28 Aug 2007 17:52:36 -0800
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: request for comments
Date: Tue, 28 Aug 2007 17:52:35 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E053A9F65@onstor-exch02.onstor.net>
In-Reply-To: <20070828183150.4e4955f1@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: request for comments
Thread-Index: Acfp3F9B2k57n1DTTu6iKKEuTY18XAAArAPg
From: "Tim Gardner" <tim.gardner@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>
Cc: "Larry Scheer" <larry.scheer@onstor.com>

Has to be in the top level makefile. Doesn't necessarily need to be in
the
rootfs and kernel makefiles. The shortcut targets just call the lower
level
makefiles with PROD=3D set to the appropriate value.

-----Original Message-----
From: Andy Sharp=20
Sent: Tuesday, August 28, 2007 6:32 PM
To: Tim Gardner
Cc: Larry Scheer
Subject: Re: request for comments

On Tue, 28 Aug 2007 18:29:24 -0700 "Tim Gardner"
<tim.gardner@onstor.com> wrote:

> Looks good Andy.
> I would like there to be the usual shortcut targets (cg-rootfs,
> bl-rootfs, etc) so that we can bypass having to specify PROD=3D on the
> make line.

That would be in the rootfs Makefile, so I'll leave that up to Larry.

> -----Original Message-----
> From: Andy Sharp=20
> Sent: Tuesday, August 28, 2007 6:20 PM
> To: Larry Scheer
> Cc: Tim Gardner
> Subject: request for comments
>=20
> OK, I think I've got the linux/kernel/Makefile hammered into a better
> fit with the make rootfs stuff.  Let me know what you think.  Here's
> the highlights:
>=20
> 'make kernel' no longer does a kernel-install
>=20
> 'make kernel-install' triggers a make kernel-build if there is no
> vmlinux file, or .config is newer than vmlinux.
>=20
> the 'make rootfs' currently does a 'make kernel-install'
>=20
> This means that if you're used to sitting in linux/kernel directory
> and doing a 'make kernel' and have it do everything including
> install, then you will have to retrain yourself to also do a 'make
> kernel-install' afterwards.
>=20
> This makes the rootfs Makefile very simple.
>=20
> Alternatively, Larry could bring back the
> $(ROOTFS)/boot/vmlinux.whatever dependency based kernel install
> portion of the rootfs Makefile, and fix it up (it wasn't ready to
> deal with vmlinux v. vmlinux.bin).
>=20
> I'll leave it up to you, Larry.  Just let me know if I need to mod the
> kernel Makefile any more.
>=20
> =
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D
>=20
> Change 25195 by andys@ripper on 2007/08/28 18:15:11 *pending*
>=20
>         Add more support for two platforms of kernels.
>         Remove kernel-install as a dependency of kernel.
>         Change kernel-install to trigger a build of the
>         kernel if there isn't a fresh one there already.
>        =20
>         Reviewed by=20
>=20
> Affected files ...
>=20
> ... //depot/dev/linux/kernel/Makefile#6 edit
