X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C80076.9F918ACF@onstor-exch02.onstor.net>; Wed, 26 Sep 2007 11:51:26 -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: PERFORCE change 25625 for review
Date: Wed, 26 Sep 2007 11:51:26 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E056C9110@onstor-exch02.onstor.net>
In-Reply-To: <20070926105857.57710c46@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 25625 for review
Thread-Index: AcgAZul3/mbqaugiRlSvUE9C1uM1IAADG5IA
From: "Larry Scheer" <larry.scheer@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Here's the rub. The developers can learn to type apt-get, make, dpkg
blah blah all day long and would be stuck with a pile of rubbish that
only produces error messages for them.

The basic issue is wget as packaged by debian doesn't cross-compile. I
fixed that. My previous changes made debian/rules work in our
cross-compile environment.

Relax, go to you doctors appointments and get your prescriptions filled.
I've got a plan.

Rendell needs to study the sources and find out what needs to change.
Once he has made his changes and is satisfied what he did works then we
create the onstor patch file that fits into the debian package
methodology and check that in to the Pkg/source tree. Now the Makefile
you set up for wget should work because we have both the onstor patch
file and the changes applied that makes wget cross-compile. I'm not
abandoning the plan we talked about. I am just getting the missing
pieces in place. We are going to need this step to create the patch
files. I am just capturing its history in perforce.

No worries,

L.

-----Original Message-----
From: Andy Sharp=20
Sent: Wednesday, September 26, 2007 10:59 AM
To: Larry Scheer
Cc: Rendell Fong
Subject: Re: PERFORCE change 25625 for review

On 25 Sep 2007 16:49:54 -0700 Larry Scheer <larry.scheer@onstor.com>
wrote:

> Change 25625 by larrys@larrys on 2007/09/25 16:43:39
>=20
> 	Exploded wget tree prior to any patches installed.=20
> 	This is the base from which the Onstor changes will be made.
> 	Previous change list #25624 added the build rules to=20
> 	cross-compile this application.
> 	Reviewed by: RendellF
>=20
> Affected files ...
>=20
> ... //depot/dev/linux/src/wget-1.10.2/AUTHORS#1 add


Larry,

I thought it was clear when we discussed this that we aren't going to
do this with wget.  Obviously it makes sense to do this with
complicated packages like glibc, but packages like exim, wget and
others, no.

Doing this specifically makes it very expensive to adopt a new version
of this package which is likely to happen 10x times more often than we
make changes to this code.

We will probably change exactly two out of 180 of these files.  Maybe
three if we have to modify the .h file.

I don't care if the developer has to spend 60 seconds learning how to
run 'apt-get source' to get the source, or doing the untar and patch by
hand.  That's 60 seconds v. 1 week to adopt a new version of the
package, v. 10 minutes to adopt a new version doing the same way exim
is done. That also goes for the other minor inconveniences caused by not
having all 180 files from the source package checked in.

I realize that this is a change from the way we have done things in the
past, and that can be discomforting, but in essence the idea here is to
fix things that didn't work for us in the past, and this is one of them.

As I said, I'm willing to create the patch and do the other work
after the developer has completed the Onstor specific changes, as
doing that might be a steeper learning curve than people want to deal
with. The makefile in the exim directory makes a good boilerplate
makefile for how to do a package like this the new way.

If either of you have any questions, feel free to seek me out.  I have
a doctor's appointment today, but I'll be in tomorrow.  If the creek
don't rise.

a
