[OmniOS-discuss] Bloody // mailwrapper & mta mediator

[Dan McDonald]

> On Mar 9, 2015, at 10:55 AM, Andy <omnios at citrus-it.net> wrote:
> 
> ; If you have suggested diffs, please mail them to the list or create webrevs.  I'm generally okay with this so long as:
> ;
> ; 	- It does not break anything else.
> ;
> ; 	- It does not hinder the post-014 goal of building illumos-gate on OmniOS.
> ;
> ; But I need to make sure.
> 
> This would be sufficient for me. It re-introduces the problem with 'mailx'
> but that was there before.
> 
> --- usr/src/pkg/manifests/SUNWcs.mf~    Mon Mar  9 14:54:01 2015
> +++ usr/src/pkg/manifests/SUNWcs.mf     Mon Mar  9 14:54:12 2015
> @@ -1871,7 +1871,3 @@
> # Depend on zoneinfo data.
> #
> depend fmri=system/data/zoneinfo type=require
> -#
> -# The mailx binary calls /usr/lib/sendmail provided by mailwrapper
> -#
> -depend fmri=system/network/mailwrapper type=require

I think this is too big of a hammer.

Tell me, would weakening the requirement of sendmail by mailwrapper help?

diff --git a/usr/src/pkg/manifests/system-network-mailwrapper.mf b/usr/src/pkg/manifests/system-network-mailwrapper.mf
index fa855da..21cc0b7 100644
--- a/usr/src/pkg/manifests/system-network-mailwrapper.mf
+++ b/usr/src/pkg/manifests/system-network-mailwrapper.mf
@@ -42,4 +42,4 @@ link path=usr/sbin/newaliases mediator=mta mediator-implementation=mailwrapper \
     target=../lib/mailwrapper
 link path=usr/sbin/sendmail mediator=mta mediator-implementation=mailwrapper \
     target=../lib/mailwrapper
-depend fmri=service/network/smtp/sendmail type=require
+depend fmri=service/network/smtp/sendmail type=optional

This keeps the spirit of the change, but doesn't trip up folks who want their own sendmail (even if they are technically violating KYSTY in their version!  ;) ).

Whatcha think?

Dan

p.s. I want to cut a bloody release today or more likely tomorrow.  Let's not bikeshed this.  I reserve the right to Just Say No also.