AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20070416201938.31d6a5d9@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	BB375AF679D4A34E9CA8DFA650E2B04E02215738@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Mon, 16 Apr 2007 20:21:08 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: cmd_upgrade.c in my home dir
Message-ID: <20070416202108.7a08baa3@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E02215738@onstor-exch02.onstor.net>
References: <20070416160848.1a597760@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E02215738@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

Since you're repeating yourself, I'll repeat myself ... no I won't.
Yes I will.  I'm stating that this is getting too complicated, and that
such a notion is ignorant of all the myriad corner cases, like the
fact that you won't be able to create a compact flash with the old
layout anymore, and so on and so forth.  There's probably ten we
haven't thought of.  In fact, this very issue is one that wasn't
thought of.  So, the story is, once you upgrade to 2.2.3, you cannot
upgrade an old flash layout any more.

a


On Mon, 16 Apr 2007 16:54:08 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> I thought this too, but Tim was talking about going from 2.2.3.0 to
> 2.2.3.1 or 2.2.5 and you don't want the new layout.
> 
> -----Original Message-----
> From: Andy Sharp 
> Sent: Monday, April 16, 2007 4:09 PM
> To: Larry Scheer
> Subject: Re: cmd_upgrade.c in my home dir
> 
> As per the discussion, I thought the consensus was that this code is
> getting too complicated, and that once the code goes into Lambo, this
> change is universal.  Keep in mind that once this code goes into
> lambo, system copy init will only create the new format.
> 
> a
> 
> 
> On Mon, 16 Apr 2007 16:04:54 -0700 "Larry Scheer"
> <larry.scheer@onstor.com> wrote:
> 
> > I checked it out, just have one question. Shouldn't the check below
> > be done in the context of version 3.X.X upgrades?
> > 
> >         /*
> >          * upgrading to old CF layout from now on is not allowed
> >          */
> >         if ((layout_x & _TO_DYN) == 0) {
> >             my_print("You are attempting to upgrade the software "
> >                 "on an old flash layout.\n");
> >             my_print("This is prohibited.  Please update layout of
> > Compact "
> >                 "Flash card with\n");
> >             my_print("the `system copy init' command first or
> > consult " "Customer Service\n");
> >             my_print("Command aborted\n");
> >             return CMD_ERR_ABORTED;
> >        
> > Where this code sits it will be run for every version. I think Tim
> > was saying we don't want this to abort if this is an upgrade for
> > something prior to a Delorean release.
> > 
> > L.
> > 
> > -----Original Message-----
> > From: Andy Sharp 
> > Sent: Monday, April 16, 2007 3:00 PM
> > To: Larry Scheer
> > Subject: cmd_upgrade.c in my home dir
> > 
> > ~andys/src/FB-DELOREAN/nfx-tree/code/ssc-nfxsh/cmd_upgrade.c
> > 
> > I believe I've fixed this up to cover the conditions we talked
> > about. It was a lot easier than I thought.  Let me know what you
> > think.
> > 
> > Cheers,
> > 
> > a
> > 
