|
On 8/2/05, Peter Mueller <pmueller@xxxxxxxxxxxx> wrote:
> > Nice patch, but you're not even checking to see if
> > /usr/lib/sendmail exists or if it even ran without errors.
> > Ideally you would make all these variables, configuration
> > variables as well. I'd have to say this patch is incomplete,
> > but it's a good idea.
>
> This is true, but it works on RedHat and Suse. I'll be happy to change my
> patch for your distro(s) if you have any patch updates..
>
> Ldirectord needs email alert functionality. I'm sure there's a better way of
> doing it - isn't there always with Perl - but this does work.
I personally would investigate getting those variables into the config
file first. And then check the return value of sendmail. You could
optionally use module as well, such as Mail::Sendmail.
-reid
|
