|Pathologically Eclectic Rubbish Lister|
Re: assert - checking up on other scriptsby mojotoad (Monsignor)
|on Jul 22, 2010 at 08:11 UTC||Need Help??|
The less you have to put into a crontab, the better. There's no reason to pass code fragments into a script via command line, much less an automated one.
You're on the right track, however, by generalizing the problem and anticipating future uses for your notification script.
The larger problem you're confronting is three things:
Once you have removed the liver from your selected recipients, your task is complete.
Seriously, though. Break the tasks down and, if not put them into their own modules, at least put them into their own subroutines.
For example, rather than passing code fragments into your script via command line, use a hash for dispatching to the correct test routine; that way you can pass a parameter called "day_stale" and not worry about the quarantine and sanitization of arbitrary code.
Finally, why not consider the unix way of things? If find -ctime 1 myfilename comes up empty, your file has been dead for a day.