Omar,

Thanks for review.

On Fri, 07 Feb 2025 23:11:48 +0100,
Omar Polo <op@omarpolo.com> wrote:
> 
> Kirill A. Korinsky <kirill@korins.ky> wrote:
> > Omar,
> > 
> > I'd like to add -bv flags to sendmail wrapper which is well known as a way
> > to validata email address.
> > 
> > Right now I need it to add a custom email to calendar(1), but it can be
> > quite useful by its own.
> > 
> > What do think?
> 
> I don't have a strong opinion on this.  I think this at least is missing
> the sendmail.8 bits though; but then if i go read postfix' sendmail(8)
> (or at least the version of the manpage available at man.archlinux.org)
> the description for -bv seems slightly different than what your diff does:
> 
> 	-bv
> 	Do not collect or deliver a message. Instead, send an email
> 	report after verifying each recipient address. This is useful for
> 	testing address rewriting and routing configurations
> 
> Have to admit that I kind of agree with what deraadt@ said in the other
> thread, if you typo the mail you don't get deliveries, even if i also
> see mvs@ point in "sendmail: command failed" not being particularly
> enlightening.
>

Interesting that an original sendmail writes to stdout status of each
recipient. So, my patch defently do not implement -bv, and I sure that 3rd
way to implement -bv is wrong.

I'd like to say sorry for noise and suggest to ignore this diff.

> Since sendmail just exit with a nonzero exit status, we maybe can get
> away without -bv and you could just log something like "calendar: failed
> to send mail to $ADDRESS" in that case.

Thanks for suggestion.

-- 
wbr, Kirill