X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C8442A.21F0E720@onstor-exch02.onstor.net>; Fri, 21 Dec 2007 16:35:12 -0700
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: agile path changes
Date: Fri, 21 Dec 2007 16:35:12 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E03B1BFC7@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: RE: agile path changes
Thread-Index: AchDfqa3rfTIfBwfS0SlUiuBhcW+ygAqv0fw
From: "Rendell Fong" <rendell.fong@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>
Cc: "Larry Scheer" <larry.scheer@onstor.com>

Is there some reason for not changing the agile directory paths being
used in the system config file list of ssc-nfxsh/verify_install.sh
script file (lines 67:71)?



-----Original Message-----
From: Andy Sharp=20
Sent: Thursday, December 20, 2007 7:08 PM
To: Larry Scheer
Cc: Rendell Fong
Subject: Re: code review

On Wed, 19 Dec 2007 17:33:39 -0800 Andrew Sharp <andy.sharp@onstor.com>
wrote:

> On Wed, 19 Dec 2007 16:57:49 -0800 "Larry Scheer"
> <larry.scheer@onstor.com> wrote:
>=20
> > Just a suggestion not sure if it is desirable in this case...
> > File: ssc-nfxsh/cmd_flash-linux.c lines: 374-377 replace the
> > system(command) with do_system() ? - up to you --
> >=20
> > Andy wants to look at these changes since he, too, is interested in
> > these changes. So I am copying him.
> >=20
> > Larry
> > _____________________________________________
> > From: Rendell Fong=20
> > Sent: Thursday, December 13, 2007 11:29 AM
> > To: Larry Scheer
> > Subject: code review
> >=20
> > Larry,
> >=20
> > Can you review these changes.  I'd like to check them in to
> > consolidate the multiple sys config file lists currently defined as
> > well as some things that need fixing for linux.

So this change has gone in now.  Let me know if you need any help
merging your changelist.

Cheers,

a

> I'd prefer that you wait until cl#26643 goes in, as these file lists
> get changed by that.  It will be much easier to merge your change on
> top of that than the other way around.  I think.
>=20
> As for things that need fixing for linux, please submit those as (a)
> separate change list?  Ie, try not to do two different things in one
> change.
>=20
> Question: have you tested these changes?  I mean, have you tried a
> 'system upgrade', a 'system config reset' and a 'system copy config'
> at the very least?  I'm sure you have.
>=20
> Also, there are several shell scripts that would be included in this
> list of files that contain config file lists.  If you're going to
> attempt to handle that, I would prefer that all that go in in one
> change, or near simultaneous changes.
>=20
> > Rendell
> >=20
> >=20
> >=20
> > Change: 26788
> >=20
> > Client: rendellf
> > User:   rendellf
> > Description:
> >     Update search path for mount/umount, fix mount options,
> > re-enable handling
> >     of snmp config files, and resetting of passwd/shadow files for
> > Linux.
> >=20
> > Files:
> >     //depot/dev/nfx-tree/code/sm-opt/cfg_files.h    # add
> >     //depot/dev/nfx-tree/code/sm-opt/fs-linux.c # edit
> >     //depot/dev/nfx-tree/code/sm-opt/fs-openbsd.c   # edit
> >     //depot/dev/nfx-tree/code/sm-opt/fs.c   # edit
> >     //depot/dev/nfx-tree/code/sm-opt/linux.h    # edit
> >     //depot/dev/nfx-tree/code/sm-opt/openbsd.h  # edit
> >     //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_flash-linux.c   # edit
> >     //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_flash-openbsd.c # edit
> >     //depot/dev/nfx-tree/code/ssc-nfxsh/cmd_upgrade.c   # edit
