AF:
NF:0
PS:10
SRH:1
SFN:
DSR:
MID:<20080226124045.3dc13048@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	BB375AF679D4A34E9CA8DFA650E2B04E056C9401@onstor-exch02.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Tue, 26 Feb 2008 12:41:26 -0800
From: Andrew Sharp <andy.sharp@onstor.com>
To: "Larry Scheer" <larry.scheer@onstor.com>
Subject: Re: please review 28044
Message-ID: <20080226124126.3e61e38c@ripper.onstor.net>
In-Reply-To: <BB375AF679D4A34E9CA8DFA650E2B04E056C9401@onstor-exch02.onstor.net>
References: <20080226111230.3deb959e@ripper.onstor.net>
	<BB375AF679D4A34E9CA8DFA650E2B04E056C9401@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

The function is only to address issues when doing upgrades from
legacy versions of 3.x, which is not relevant to Linux.

On Tue, 26 Feb 2008 11:54:18 -0800 "Larry Scheer"
<larry.scheer@onstor.com> wrote:

> nfx-tree/code/ssc-nfxsh/verify_install.in
> 
>      fix_old_paths() has nothing for Linux systems to do and it did
>      previously. (Maybe that was unnecessary.) I am not 100% sure of
>      this but doesn't the shell carp about empty functions? It is not
>      exactly empty because it has to do the test. Just musing...
> 
> -----Original Message-----
> From: Andy Sharp 
> Sent: Tuesday, February 26, 2008 11:13 AM
> To: Larry Scheer
> Subject: Re: please review 28044
> 
> Ping
> 
> On Mon, 25 Feb 2008 15:18:42 -0800 Andrew Sharp
> <andy.sharp@onstor.com> wrote:
> 
> > OK,
> > 
> > I've added those changes too.  Stupid BSD.
> > 
> > Change 28044 by andys@ripper on 2008/02/25 08:32:31 *pending*
> > 
> >         TED00022254 - at what point do we
> > zip /var/log/onstor/messages TED00022350 - Getting message "Usage:
> > mktemp..." 
> >         Make sure upgrade properly changes newsyslog.conf.
> >         Fix calls to OpenBSDs mktemp utility.
> >         
> >         reviewed by
> > 
> > Affected files ...
> > 
> > ... //depot/dev/nfx-tree/code/ssc-nfxsh/verify_install.in#3 edit
> > 
> > 
> > 
> > On Mon, 25 Feb 2008 14:45:05 -0800 "Larry Scheer"
> > <larry.scheer@onstor.com> wrote:
> > 
> > > nfx-tree/code/ssc-nfxsh/verify_install.in
> > > 
> > >      Line 911: I know you didn't change this line but mktemp
> > > doesn't work as written in this script on BSD systems. There is a
> > > defect filed for this (TED00022350). Might as well fix it while
> > > you are in this routine.
> > > 
> > >      Line 177: Has the same problem with mktemp usage on BSD
> > > systems.
> > > 
> > >      Who new mktemp wasn't the same between OpenBSD and Linux?
> > > 
> > > 
> > > -----Original Message-----
> > > From: Andy Sharp 
> > > Sent: Monday, February 25, 2008 1:34 PM
> > > To: Larry Scheer
> > > Subject: please review 28044
> > > 
> > > Change 28044 by andys@ripper on 2008/02/25 08:32:31 *pending*
> > > 
> > >         TED00022254 - at what point do we
> > > zip /var/log/onstor/messages 
> > >         Make sure upgrade properly changes newsyslog.conf.
> > >         
> > >         reviewed by
> > > 
> > > Affected files ...
> > > 
> > > ... //depot/dev/nfx-tree/code/ssc-nfxsh/verify_install.in#3 edit
> > > 
