AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20071227233735.609102e1@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	BB375AF679D4A34E9CA8DFA650E2B04E042F0007@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Thu, 27 Dec 2007 23:38:09 -0800
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: Code review request
Message-ID: <20071227233809.3f84bf3d@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E042F0007@onstor-exch02.onstor.net>
References: <BB375AF679D4A34E9CA8DFA650E2B04E056C9269@onstor-exch02.onstor.net>
	<20071227184510.03ed0747@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E042F0006@onstor-exch02.onstor.net>
	<20071227224615.55abc135@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E042F0007@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

On Thu, 27 Dec 2007 23:10:58 -0800 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> Yup uname gives the same results as uname -s on both Linux and
> OpenBSD try it sometime!
> 
> Yea, I still have to create the link.It is the cleanest solution.
> Creating the files will mess with the BOM file in a way I don't want.
> The link will exist on 3.X systems anywat -- we never remove it with
> any install. Unless we do a BSD release for 4.0 the link won't exist
> on 4.0 systems. The only other thing I can think of is have
> verify_install remove the link at the end. Not real clean and I
> wonder if it is worth the effort.
> 
> Thanks for the review 

NP

Hey, if you're at work tomorrow, could you do me a favor?  Could you
stick a spare CF card in the second slot of bobcat g7r9?  It is empty.
I need to test all my horrible hacks to flash_install.sh on BSD.  It
seems to work pretty well-ish on Linux now.

Cheers,

a

> Larry
> 
> 
> -----Original Message-----
> From: Andy Sharp
> Sent: Thu 12/27/2007 10:46 PM
> To: Larry Scheer
> Subject: Re: Code review request
>  
> On Thu, 27 Dec 2007 22:01:14 -0800 "Larry Scheer"
> <larry.scheer@onstor.com> wrote:
> 
> > Ok I removed the Host line in my client. You should be able to
> > access it from ripper.
> > 
> > 
> > 
> > -----Original Message-----
> > From: Andy Sharp
> > Sent: Thu 12/27/2007 6:45 PM
> > To: Larry Scheer
> > Subject: Re: Code review request
> >  
> > Oooh, close.  I get
> > Client 'larrys-r14-dmip' can only be used from host
> > 'linux-compile.onstor.net'.
> > 
> > I should be able to do it from linux-compile, but there is some
> > weird problem happening where remote X windows aren't working from
> > linux-compile today.  You should just be able to adjust the client
> > and remove the host field value.  IIRC.
> > 
> > On Thu, 27 Dec 2007 17:21:06 -0800 "Larry Scheer"
> > <larry.scheer@onstor.com> wrote:
> > 
> > > Andy,
> > >      My workspace is mounted on ripper as:
> > > /home/larrys/perforce/trees/dev
> > > So you should be able to diff these file from ripper.
> > > 
> > > This fixes the "broken system upgrade problem"
> > > 
> > > Verify_install.sh is still a work in progress. The changes here I
> > > am most interested in getting in are the path changes. I probably
> > > have much more work to do using md5sum on the Linux side. I just
> > > cobbled some changes together that switch between cksum and
> > > md5sum, for now. Right now I want to make sure I didn't break the
> > > BSD stuff.
> > > 
> > > I have another change list coming that fixes the descr files
> > > (once I test my fixes.)
> > > 
> > > Change 27166 by larrys@larrys-r14-dmip on 2007/12/27 16:12:58
> > > *pending*
> > > 
> > >         Add compatibility with older versions of cmd_upgrade code 
> > >         attempting to upgrade to the newer system "sans agile."
> > >         Reviewed by: AndyS
> > > 
> > > Affected files ...
> > > 
> > > ... //depot/dev/nfx-tree/Makefile#6 edit
> > > ... //depot/dev/nfx-tree/code/ssc-nfxsh/verify_install.sh#4 edit
> > > ... //depot/dev/nfx-tree/openbsd.mk#4 edit
> 
> 
> 
> nfx-tree/Makefile
> 
>      looks good
> 
> nfx-tree/code/ssc-nfxsh/verify_install.sh
> 
>      heh-heh line 44, does that work?  i would assume that you have
>      to do uname -s at least.
> 
>      rest is fine
> 
> nfx-tree/openbsd.mk
> 
>      so basically what you're saying is we still have to create the
>      compat symlink?  can't we just put those files there, no symlink?
>      create a /usr/local/agile/bin/verify_install.sh and whatever?
> then the upgrade program will execute /usr/local/bin/verify_install.sh
>      ...  ferk.  nm.
> 
> 
