X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C882D0.5DCD09FE@onstor-exch02.onstor.net>; Mon, 10 Mar 2008 10:01:21 -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: please review 28222
Date: Mon, 10 Mar 2008 10:01:21 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E03B1C0DD@onstor-exch02.onstor.net>
In-Reply-To: <20080310093147.3cf5beda@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: please review 28222
Thread-Index: AciCzDx8ORTvKePvQMm9bGyAJHHHjgAA+ZXg
From: "Rendell Fong" <rendell.fong@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Looks ok.


-----Original Message-----
From: Andy Sharp=20
Sent: Monday, March 10, 2008 9:32 AM
To: Rendell Fong
Subject: please review 28222

Hi Rendell,

I sent this one to you before, but then all the controversy with
crontab header spammage got added to it, so I added changes to these
three other files besides emrscron.  The big thing is the modify-cron
program itself.  Let me know what you think.

Cheers,

a



Change 28222 by andys@ripper on 2008/03/06 20:12:21 *pending*

        Fix several emrscron/modify-cron confusions and crontab
        file header line spammage that occurs in Linux.
       =20
        Fix spurious error messages on Linux from emrscron
        about newsyslog and killing itself and so on.
       =20
        Also get rid of a huge pustule of trailing whitespace
        in emrscron.
       =20
        reviewed by

Affected files ...

... //depot/dev/nfx-tree/code/ssc-modify-cron/modify-cron.c#2 edit
... //depot/dev/nfx-tree/code/ssc-nfxsh/verify_install.in#6 edit
... //depot/dev/nfx-tree/code/ssc-pm/rc.onstor#1 edit
... //depot/dev/nfx-tree/code/ssc-support/emrscron#16 edit

