[PATCHSET] Factor out common KDC infrastructure
Jeremy Allison
jra at samba.org
Fri Jun 17 19:23:35 UTC 2016
On Fri, Jun 17, 2016 at 10:33:49AM +0200, Volker Lendecke wrote:
> On Thu, Jun 16, 2016 at 04:54:06PM +0200, Andreas Schneider wrote:
> > On Thursday, 16 June 2016 16:50:37 CEST Andreas Schneider wrote:
> > > Hello,
> >
> > Now with a patch :)
>
> Looks very good from my point of view. If I had done more in the KDC
> area, I'd give you a Reviewed-by.
I fear nothing ! :-).
I won't get into the politics of it, or merging back from
newer Heimdal versions (*are* there newer versions ?) but
the code changes look clear and clean.
One extra change was required for build though - I don't
have ldb installed outside of samba on my system so it
also requires:
diff --git a/source4/kdc/wscript_build b/source4/kdc/wscript_build
index 3182b73..762e433 100755
--- a/source4/kdc/wscript_build
+++ b/source4/kdc/wscript_build
@@ -48,6 +48,7 @@ bld.SAMBA_SUBSYSTEM('KDC-SERVER',
includes=kdc_include,
deps='''
krb5samba
+ ldb
LIBTSOCKET
LIBSAMBA_TSOCKET
''',
in order for source4/kdc/kdc-proxy.c to build (otherwise
it can't find ldb.h).
Reviewed-by: Jeremy Allison <jra at samba.org>
Pushed.
More information about the samba-technical
mailing list