[PATCHSET]: let configure(waf) fail if --with-ads was specified but ad support was not found
Michael Adam
obnox at samba.org
Fri Nov 2 06:59:08 MDT 2012
On 2012-11-02 at 23:26 +1100, Andrew Bartlett wrote:
>
> You are not missing anything. Back in May or so we talked about having
> Samba require krb5, ldap and ACL headers by default, so users would not
> be surprised by the lack of these features in compiled binaries. It
> would make --with-ads the default, and require an explicit --without-ads
> if you didn't want it.
Ah, now I understand your point. :-)
The above would be achieved by failing configure if "with_ads" was
set to True and support was not found. And of course letting the default
at "true" where it currently is.
The proposed patchset does a bit more: It lets configure fail
if --with-ads is specified, but it adds the None default, which
is effectively the auto mode, thereby _not_ changing the default
behaviour.
Not reading back the discussions, I think the default to "auto"
is quite reasonable. If you don't specify anyhting, you'll get
what is there. If you want to enforce it (like packagers will),
you specify --with-ads, and then you don't get surprises.
This is exactly the mode that distributors are used to from
the s3 autoconf build.
What do you think?
Thanks - Michael
> (and the same for ACL support).
>
> It even got as far as a promise in BUILD_SYSTEMS.txt, but I never
> managed to do the code. I was trying to see if you might take it on.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 206 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20121102/5e3bc920/attachment.pgp>
More information about the samba-technical
mailing list