Beefy Boxes and Bandwidth Generously Provided by pair Networks
"be consistent"

Re: Crond

by osfameron (Hermit)
on May 21, 2002 at 13:46 UTC ( #168122=note: print w/replies, xml ) Need Help??

in reply to Crond

A few comments after a quick scan through the code:
  • I'm not sure how you are supposed to install this as a Win32 service (The supplied subs, "Remove", "Install", etc. don't seem to have anything pointing to them).
  • Some of this code looks like it should be refactored: e.g. the code for minutos and horas are almost identical, and with some re-writing you could have much simpler code.
  • You're not useing strict or warnings!
  • The name of the crontab file is hard-coded in the script. You should probably document that!
  • (And you should probably document that the names of the days are hard-coded in Spanish, and where to change them. If you've not learned how to use POD documentation try reading the perlpod manpage, very useful!)
Hope that helps.


Replies are listed 'Best First'.
Re: Re: Crond
by zeroquo (Novice) on May 21, 2002 at 14:34 UTC
    The code its make for Perl Dev Kit of ActiveState, its make an binary distribution of this code as native W32 service. Its announce, the only donīt use the Strict package and Warning package, because the module Perlsvc its fail on compile the source. That work similar that use Strict package, whit the use of "my" attribute for variable definition. Probe that , generating the binary file whith perlsvc compile package of ActiveState. Enjoy & Thanks !

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://168122]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others making s'mores by the fire in the courtyard of the Monastery: (7)
As of 2018-07-17 21:34 GMT
Find Nodes?
    Voting Booth?
    It has been suggested to rename Perl 6 in order to boost its marketing potential. Which name would you prefer?

    Results (378 votes). Check out past polls.