AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20070926171909.789e6b9b@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<larry.scheer@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#imap/andys@onstor.net@onstor-exch02.onstor.net/INBOX	0	BB375AF679D4A34E9CA8DFA650E2B04E056C9110@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 26 Sep 2007 17:19:26 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: PERFORCE change 25625 for review
Message-ID: <20070926171926.76dcd721@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E056C9110@onstor-exch02.onstor.net>
References: <20070926105857.57710c46@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E056C9110@onstor-exch02.onstor.net>
Organization: Onstor
X-Mailer: Sylpheed-Claws 2.6.0 (GTK+ 2.8.20; x86_64-pc-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Sorry Larry, you're absolutely right, I think I was just a bit nervous
about my Dr. appointment, but at least that news is good-ish: no
surgery indicated for the moment on my neck; MRI needed on my shoulder;
stupid doctor didn't even seem to notice that my mid and lower back
also had fractured vertebrae and seemed almost completely unconcerned
about that.  So maybe I should be too.

As you stated, since all that mmmfsfammsaf is already checked in, it's
steady as she goes until the developer (Rendell I'm assuming) has got
his mods made and tested.  Then {me,you if you want} can get things back
more towards the grand scheme.  I'm trying to listen carefully to what
you are telling me and take it into account.  I forgot that I have some
un-checked in mods to the exim makefile that do developer things, like
unpack all the source and apply patches and sh!t like that so someone
can just do 'make source' for example and get everything they need.
But I see that I still need to go farther down that road to make this
plan more generally useable, so I'll work on that some more.

I realize the first stuff you did was to get the xcompile working, but
when I saw the other checkin, I'm like 'Oh nooooooooo....' [Mr. Bill]
Next time I'll know to do a bit more than just check in the tar balls
before trying to hand it off.

Good going,

a


On Wed, 26 Sep 2007 12:51:26 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> 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 
> 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
> > 
> > 	Exploded wget tree prior to any patches installed. 
> > 	This is the base from which the Onstor changes will be made.
> > 	Previous change list #25624 added the build rules to 
> > 	cross-compile this application.
> > 	Reviewed by: RendellF
> > 
> > Affected files ...
> > 
> > ... //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
