AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20080407170326.6cbb8f3b@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	BB375AF679D4A34E9CA8DFA650E2B04E056C9536@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Mon, 7 Apr 2008 17:03:43 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: please review 28660 - should be a quickie
Message-ID: <20080407170343.2763f205@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E056C9536@onstor-exch02.onstor.net>
References: <20080407155033.5f07e017@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E056C9536@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

You must have been thinking of your proposal when the phrase "...make no
sense" came to mind.  The only thing I had to change, if I understand
your point, was the removal of the Images directory, which, hey, I
don't know, I was just guessing.  That's why I sent it to you for
reviewing.

Fixed.  As long as the only thing that needed fixing was getting rid of
the Images directory.


On Mon, 7 Apr 2008 16:24:17 -0700 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> Lines 165 through 173 make no sense. On a cheetah ssc-mgmt.bin is
> located in $1/ssc-mgmt.bin not in Images. There is no ssc-mgmt.bin
> anywhere for bobcat.
> 
> So I think you meant to write:
> 
> if [ `echo /*bc.bin | wc -c` = 3] ; then
>  # if on a bobcat, don't install cheetah distro
>  if [ $1/ssc-mgmt.bin ] ; then
>     return 1
>  fi
> # must be on a cheetah ?
> elif [ ! -e $1/ssc-mgmt.bin ] ; then
>     return 1
> fi 
> 
> -----Original Message-----
> From: Andy Sharp 
> Sent: Monday, April 07, 2008 3:51 PM
> To: Larry Scheer
> Subject: Re: please review 28660 - should be a quickie
> 
> Alright, added some code, but since I don't have a cheetah, I don't
> know if it's right.  So please re-review.
> 
> 
> On Fri, 4 Apr 2008 19:54:56 -0700 "Larry Scheer"
> <larry.scheer@onstor.com> wrote:
> 
> > nfx-tree/Tools/flash_install.sh
> > 
> >      This will prevent a cheetah or bobcat build from being
> > installed on a cougar.  But it won't stop a cheetah release from
> > being installed on a bobcat. Which is fine if that was all you
> > wanted to at this time
> > 
> >      If you want to go one step further a cheetah has this file:
> >      /ssc-mgmt.bin and bobcats do not. You could test for the
> > existence of ssc-mgmt.bin if you know you are running on a cheetah
> > or a bobcat.
> > 
> > -----Original Message-----
> > From: Andy Sharp 
> > Sent: Thursday, April 03, 2008 2:36 PM
> > To: Larry Scheer
> > Subject: please review 28660 - should be a quickie
> > 
> > A quickie, hopefully:
> > 
> > Change 28660 by andys@ripper on 2008/04/03 14:34:48 *pending*
> > 
> >         Bolster sanity check that should prevent installing BSD
> > onto a Linux based system, and vice-versa.
> >         
> >         reviewed by
> > 
> > Affected files ...
> > 
> > ... //depot/dev/nfx-tree/Tools/flash_install.sh#11 edit
> > 
