[PATCH] Add MDNS configuration option

Andrew Bartlett abartlet at samba.org
Tue Dec 5 22:32:16 UTC 2017


On Tue, 2017-12-05 at 17:26 -0500, Kevin Anderson wrote:
> Hi Andrew,
> 
> > > > > Can you give more of a rationale for why this is needed?
> > > > 
> > > > <https://lists.samba.org/archive/samba-technical/2017-October/123231.html>
> > > > <https://lists.samba.org/archive/samba-technical/2017-November/123667.html>
> > > 
> > > I can provide additional clarification if required but those two
> > > threads sum things up in my opinion.
> > 
> > Thanks.  Can you just put that text into the commit?  It makes it
> > easier for us to remember the details later.  Also please update the
> > WHATSNEW.txt so the release announcement will include it. 
> 
> I've updated WHATSNEW for both the addition of Time Machine support as well
> as the MDNS change. I felt like adding the Time Machine note was appropriate
> but if you would like me to remove it just let me know. I've also updated the
> commit to include the reason behind the change as well. I tried to keep it
> brief but if you'd like to see more information I can update that as well.

Reviewed-by: Andrew Bartlett <abartlet at samba.org>

Can I get a second team reviewer please?

Thanks,

Andrew Bartlett

-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba







More information about the samba-technical mailing list