X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C810EA.7BF3E1AE@onstor-exch02.onstor.net>; Wed, 17 Oct 2007 10:21:06 -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: Please review
Date: Wed, 17 Oct 2007 10:21:06 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E056C9144@onstor-exch02.onstor.net>
In-Reply-To: <20071017111851.4c3b7ea7@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: Please review
Thread-Index: AcgQ6iwtGSHCdXptQGSi2cJjxJ1/xwAAB5jw
From: "Larry Scheer" <larry.scheer@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Okay, no problem. Those changes can wait. Now that I have the .deb built
I am going to test the build and configure of a rootfs with the new
package.

Larry

-----Original Message-----
From: Andy Sharp=20
Sent: Wednesday, October 17, 2007 11:19 AM
To: Larry Scheer
Subject: Re: Please review

On Wed, 17 Oct 2007 11:10:33 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> Are you going to be in today? I would like to explain the changes to
> the Makefile. I broke up the one main target into chunks to facilitate
> debugging and the case where a patch fails.
>=20
> I can move the patch file from the src directory to the source
> hierarchy. No big thing there.
>=20
> Any way it would be good for us to go over these changes in detail
> together and not via email.

I completely agree which is why I cut the comments short.  Also there
were a lot of changes and I figured why go over all this bit-by-bit in
my head and try and figure it out when 10 minutes with Larry will take
care of it.  Unfortunately, it doesn't look good for me coming in today,
even though I originally planned on it.  I had to take a pain pill this
morning and I can't drive after taking one of those howizters.  ~:^(

> Larry=20
>=20
> -----Original Message-----
> From: Andy Sharp=20
> Sent: Tuesday, October 16, 2007 6:09 PM
> To: Larry Scheer
> Subject: Re: Please review
>=20
> On Tue, 16 Oct 2007 17:39:03 -0700 "Larry Scheer"
> <larry.scheer@onstor.com> wrote:
>=20
> > My workspace is ~larrys/src/dev
> >=20
> > Change 25873 by larrys@larrys on 2007/10/16 12:32:38 *pending*
> >=20
> >         Makefile updated with rules to patch debian code and update
> > the version in the changelog with an epoch number.
> >         Reviewed by: AndyS
> >=20
> > Affected files ...
> >=20
> > ... //depot/dev/linux/Pkgs/source/wget/Makefile#1 edit
> > ... //depot/dev/linux/src/wget-1.10.2/debian/changelog#1 edit
> > ...
> //depot/dev/linux/src/wget-1.10.2/debian/patches/debianrules.patch#1
> > add
> > ...
> //depot/dev/linux/src/wget-1.10.2/debian/patches/onstor-src.dpatch#1
> > add
>=20
> linux/Pkgs/source/wget/Makefile
>=20
>      wah?  we need to go over this in detail.
>=20
>=20
> linux/src/wget-1.10.2/debian/changelog
>=20
>      looks good
>=20
>=20
> linux/src/wget-1.10.2/debian/patches/debianrules.patch
>=20
>      >>add linux/src/wget-1.10.2/debian/patches/debianrules.patch
>=20
>      uh, huh?  linux/src is the wrong directory
>=20
>=20
> linux/src/wget-1.10.2/debian/patches/onstor-src.dpatch
>=20
>      >>add linux/src/wget-1.10.2/debian/patches/onstor-src.dpatch
>=20
>      nope, all this stuff needs to go in Source, remember?  we are
>      doing away with src.  we have Pkgs/binary and Pkgs/source, no
>      additional stuff in linux/src.
>=20
>=20
