AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20070926105711.6a7fa63f@ripper.onstor.net>
CFG:
PT:0
S:andy.sharp@onstor.com
RQ:
SSV:onstor-exch02.onstor.net
NSV:
SSH:
R:<larry.scheer@onstor.com>,<rendell.fong@onstor.com>
MAID:1
X-Sylpheed-Privacy-System:
X-Sylpheed-Sign:0
SCF:#mh/Mailbox/sent
RMID:#mh/Mailbox/perfarce	0	WEBMAILXBKDslFeQv7K00000243@mail.onstor.com
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 26 Sep 2007 10:58:57 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: Larry Scheer <larry.scheer@onstor.com>
Cc: Rendell Fong <rendell.fong@onstor.com>
Subject: Re: PERFORCE change 25625 for review
Message-ID: <20070926105857.57710c46@ripper.onstor.net>
In-Reply-To: <WEBMAILXBKDslFeQv7K00000243@mail.onstor.com>
References: <WEBMAILXBKDslFeQv7K00000243@mail.onstor.com>
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

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
