I'd forgotten that we had completed the task of getting rid of ns_sendmail calls cleaned out of all of core and dotlrn. I remembered working on replacing some of the calls with calls to acs_mail_lite::send, but didn't remember that we'd finished. We did good, then :)
Alternative 2 has my vote, too, it's how I'd imagined implementing daveb's suggestion (use acs_mail_lite::send, and thus use the package parameters).
Sending the warning to the log is a good idea, yes.
