X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C840DD.7A5DBE64@onstor-exch02.onstor.net>; Mon, 17 Dec 2007 10:48:56 -0800
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 26965 for review
Date: Mon, 17 Dec 2007 10:48:56 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E072017AA@onstor-exch02.onstor.net>
In-Reply-To: <20071217104001.03ddb6e5@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 26965 for review
Thread-Index: AchA3Dt6LqM45BPjTkyet0JXLZrfEAAARP+Q
From: "Ramesh R A. Andavar" <randavar@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>
Cc: "Ed Kwan" <ed.kwan@onstor.com>

Andy,

1 file is affected for this fix.

nfx-tree/code/ssc-nfxsh/verify_install.sh

Regards
Ramesh

-----Original Message-----
From: Andy Sharp=20
Sent: Monday, December 17, 2007 10:40 AM
To: Ramesh R A. Andavar
Cc: Ed Kwan
Subject: Re: PERFORCE change 26965 for review

I would need the changelist #.  Thanks.

On Mon, 17 Dec 2007 10:31:41 -0800 "Ramesh R A. Andavar"
<randavar@onstor.com> wrote:

> Hi Andy,
>=20
> Please review the code as soon as possible and let me know if there is
> any defect. This bug is a must fix bug for R98. I should do a check-in
> by today.=20
>=20
> My P4 client: ramesh_DEV
>=20
> Regards
> Ramesh
>=20
> -----Original Message-----
> From: Ed Kwan=20
> Sent: Monday, December 17, 2007 10:26 AM
> To: Ramesh R A. Andavar; Andy Sharp
> Subject: RE: PERFORCE change 26965 for review
>=20
> Fine with me, but you should check with Andy.
>=20
> > -----Original Message-----
> > From: Ramesh R A. Andavar
> > Sent: Monday, December 17, 2007 9:25 AM
> > To: Andy Sharp; Ed Kwan
> > Subject: RE: PERFORCE change 26965 for review
> >=20
> > Yes, I was wrong. I changed the code. Please review it once again.
> >=20
> > Regards
> > Ramesh
> >=20
> > -----Original Message-----
> > From: Andy Sharp
> > Sent: Friday, December 14, 2007 6:27 PM
> > To: Ed Kwan
> > Cc: Ramesh R A. Andavar
> > Subject: Re: PERFORCE change 26965 for review
> >=20
> > On Fri, 14 Dec 2007 17:16:48 -0800 "Ed Kwan" <ed.kwan@onstor.com>
> wrote:
> >=20
> > > You code is:
> > >   ln -s /etc/localtime /usr/share/zoneinfo/US/Pacific
> > >
> > > Is the direction of the link correct?
> >=20
> > That would be "no" ~:^)
> >=20
> > >
> > > > -----Original Message-----
> > > > From: Ramesh R A. Andavar
> > > > Sent: Friday, December 14, 2007 5:08 PM
> > > > To: Ed Kwan; Andy Sharp
> > > > Subject: RE: PERFORCE change 26965 for review
> > > >
> > > >
> > > > I have Changed the code to use test -r. Please review it again.
> > > >
> > > > Regards
> > > > Ramesh
> > > >
> > > > -----Original Message-----
> > > > From: Ed Kwan
> > > > Sent: Friday, December 14, 2007 4:47 PM
> > > > To: Ramesh R A. Andavar; Andy Sharp
> > > > Subject: RE: PERFORCE change 26965 for review
> > > >
> > > > Instead of using dd to see if the file exists, will "test -r" do
> the
> > > > trick?
> > > >
> > > > > -----Original Message-----
> > > > > From: Ramesh R A. Andavar
> > > > > Sent: Friday, December 14, 2007 3:44 PM
> > > > > To: Ed Kwan; Andy Sharp
> > > > > Subject: RE: PERFORCE change 26965 for review
> > > > >
> > > > > Hi Ed/Andy,
> > > > >
> > > > > I have changed the code to check whether /etc/localtime points
> to
> > > > > an invalid file.
> > > > >
> > > > > Please review my code and let me know your comments
> > > > >
> > > > > My P4 client is ramesh_DEV
> > > > >
> > > > > Regards
> > > > > Ramesh.
> > > > >
> > > > > -----Original Message-----
> > > > > From: Ed Kwan
> > > > > Sent: Friday, December 14, 2007 1:27 PM
> > > > > To: Andy Sharp; Ramesh R A. Andavar
> > > > > Subject: RE: PERFORCE change 26965 for review
> > > > >
> > > > > Interesting point.
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Andy Sharp
> > > > > > Sent: Friday, December 14, 2007 11:34 AM
> > > > > > To: ramesh
> > > > > > Cc: Ed Kwan
> > > > > > Subject: Re: PERFORCE change 26965 for review
> > > > > >
> > > > > > On 14 Dec 2007 11:06:55 -0800 ramesh <ramesh@build-k2-depot>
> > > wrote:
> > > > > >
> > > > > > > Change 26965 by ramesh@ramesh_DEV on 2007/12/14 10:59:41
> > > > > > >
> > > > > > > 	On upgrade the time zone is reset to PST.
> > > > > > > Modified the upgrade script to skip the /etc/localtime.
> > > > > > > TED00020604.
> > > > > > >
> > > > > > > 	Reviewed by Ed Kwan
> > > > > > >
> > > > > > > Affected files ...
> > > > > > >
> > > > > > > ...
//depot/dev/nfx-tree/code/ssc-nfxsh/verify_install.sh#2
> > > > > > > edit
> > > > > > >
> > > > > > >
> > > > > > > =
http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/26965?ac=3D10
> > > > > >
> > > > > > Hi guys,
> > > > > >
> > > > > > This change is not sufficient.  If you want to do this, more
> > > > > > logic
> > > has
> > > > > > to be added, because the timezone files might change on an
> > > upgrade,
> > > > > > leaving /etc/localtime pointing to nothing.  So you can skip
> it
> > > here,
> > > > > > but after the upgrade is over, you have to check to see if
> > > > > > it
> > > still
> > > > > > points to something, and if it doesn't, reset it to PST.
