Re: [Stp-devel] [PATCH] stp_cron.pl simplifications (2/3)

[Mark W. <ma...@os...>]

On Thu, Apr 28, 2005 at 04:44:40PM -0700, Cliff White wrote:
> > --Multipart_Thu__28_Apr_2005_15_20_23_-0700_Idruq+WvDTP1_vto
> > Content-Type: text/plain; charset=US-ASCII
> > Content-Transfer-Encoding: 7bit
> > Content-Disposition: inline
> > 
> > On Thu, 31 Mar 2005 08:56:53 -0800
> > Mark Wong <ma...@os...> wrote:
> > 
> > >    added --verbose and --no_action options for debugging messages
> > >    with the option of not committing any changes
> > > 
> > >    removed additional scripts, one of which was almost a duplication,
> > >    that were for debugging without committing changes
> > 
> > Need to revisit this patch.  After learning some more perl-isms, I
> > pulled out the BEGIN function because I can't have any command line
> > interface flags take affect until after BEGIN is executed.  Otherwise,
> > any see a problem with the way I've added the --no_action logic?
> 
> So far, no, put you should remove the mention of BEGIN in the lockfile exit printf, since BEGIN is a resevered term for perl.

Ok, will do.  Thanks.

Mark