AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20070416160839.2432a038@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	BB375AF679D4A34E9CA8DFA650E2B04E02215737@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Mon, 16 Apr 2007 16:08:48 -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: <20070416160848.1a597760@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E02215737@onstor-exch02.onstor.net>
References: <20070416150006.6507cd1e@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E02215737@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

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
> 
