The branch, master has been updated via 05c09e8cfa0 heimdal_build: Prepare for Heimdal upgrade by only building HEIMDAL_ASN1_GEN_HOSTCC when needed. via 98cb41cb35d build: Remove kdc_include except where needed via 209a33670fa build: Only use embedded Heimdal include paths in an embedded Heimdal build from d6380560f87 docs: fix documentation for default of "fruit:zero_file_id"
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 05c09e8cfa09d22b31b7da6b461413dfb807984a Author: Andrew Bartlett <abart...@samba.org> Date: Thu Dec 2 13:25:07 2021 +1300 heimdal_build: Prepare for Heimdal upgrade by only building HEIMDAL_ASN1_GEN_HOSTCC when needed. This will otherwise break the system-heimdal build. This is correct regardless. Signed-off-by: Andrew Bartlett <abart...@samba.org> Reviewed-by: Joseph Sutton <josephsut...@catalyst.net.nz> Autobuild-User(master): Andrew Bartlett <abart...@samba.org> Autobuild-Date(master): Mon Dec 6 21:48:30 UTC 2021 on sn-devel-184 commit 98cb41cb35dfacbd5c6acfb13a0ac555b474da08 Author: Andrew Bartlett <abart...@samba.org> Date: Thu Dec 2 11:47:35 2021 +1300 build: Remove kdc_include except where needed This include was being set on too many subsystems, including some MIT-related. This was a problem because it would then trigger the mixing of MIT and Heimdal krb5.h files. It is now only set on the plugins and services that use the embedded Heimdal KDC. BUG: https://bugzilla.samba.org/show_bug.cgi?id=14924 Signed-off-by: Andrew Bartlett <abart...@samba.org> Reviewed-by: Joseph Sutton <josephsut...@catalyst.net.nz> commit 209a33670fab5dd7373444ae1ce76dbb5dfa0058 Author: Andrew Bartlett <abart...@samba.org> Date: Thu Dec 2 11:33:02 2021 +1300 build: Only use embedded Heimdal include paths in an embedded Heimdal build BUG: https://bugzilla.samba.org/show_bug.cgi?id=14924 Signed-off-by: Andrew Bartlett <abart...@samba.org> Reviewed-by: Joseph Sutton <josephsut...@catalyst.net.nz> ----------------------------------------------------------------------- Summary of changes: buildtools/wafsamba/samba3.py | 4 ++-- source4/heimdal_build/wscript_build | 18 +++++++++--------- source4/kdc/wscript_build | 9 --------- 3 files changed, 11 insertions(+), 20 deletions(-) Changeset truncated at 500 lines: diff --git a/buildtools/wafsamba/samba3.py b/buildtools/wafsamba/samba3.py index ebc7fbb707f..4277c5f6f2e 100644 --- a/buildtools/wafsamba/samba3.py +++ b/buildtools/wafsamba/samba3.py @@ -35,8 +35,8 @@ def s3_fix_kwargs(bld, kwargs): # the extra_includes list is relative to the source3 directory extra_includes = [ '.', 'include', 'lib' ] - # local heimdal paths only included when USING_SYSTEM_KRB5 is not set - if not bld.CONFIG_SET("USING_SYSTEM_KRB5"): + # local heimdal paths must only be included when using our embedded Heimdal + if bld.CONFIG_SET("USING_EMBEDDED_HEIMDAL"): extra_includes += [ '../source4/heimdal/lib/com_err', '../source4/heimdal/lib/krb5', '../source4/heimdal/lib/gssapi', diff --git a/source4/heimdal_build/wscript_build b/source4/heimdal_build/wscript_build index 079cac744f9..77519356575 100644 --- a/source4/heimdal_build/wscript_build +++ b/source4/heimdal_build/wscript_build @@ -856,21 +856,21 @@ HEIMDAL_SUBSYSTEM('HEIMDAL_VERS_HOSTCC', use_global_deps=False, use_hostcc=True) -HEIMDAL_SUBSYSTEM('HEIMDAL_ASN1_GEN_HOSTCC', - 'lib/asn1/gen.c', - includes='../heimdal/lib/asn1', - group='hostcc_build_main', - cflags=bld.env.HEIMDAL_UNPICKY_WNO_STRICT_OVERFLOW_CFLAGS, - deps='ROKEN_HOSTCC', - use_global_deps=False, - use_hostcc=True) - HEIMDAL_SUBSYSTEM('HEIMDAL_VERS', 'lib/vers/print_version.c ../heimdal_build/version.c', deps='roken replace') if not bld.CONFIG_SET('USING_SYSTEM_ASN1_COMPILE'): + HEIMDAL_SUBSYSTEM('HEIMDAL_ASN1_GEN_HOSTCC', + 'lib/asn1/gen.c', + includes='../heimdal/lib/asn1', + group='hostcc_build_main', + cflags=bld.env.HEIMDAL_UNPICKY_WNO_STRICT_OVERFLOW_CFLAGS, + deps='ROKEN_HOSTCC', + use_global_deps=False, + use_hostcc=True) + # here is the asn1 compiler build rule HEIMDAL_BINARY('asn1_compile', 'lib/asn1/gen_copy.c ' diff --git a/source4/kdc/wscript_build b/source4/kdc/wscript_build index 0edca94e75f..c7f28a72342 100644 --- a/source4/kdc/wscript_build +++ b/source4/kdc/wscript_build @@ -58,7 +58,6 @@ bld.SAMBA_LIBRARY('HDB_SAMBA4', bld.SAMBA_LIBRARY('HDB_SAMBA4_PLUGIN', source='hdb-samba4-plugin.c', deps='hdb HDB_SAMBA4 samba-util samba-hostconfig ', - includes=kdc_include, link_name='modules/hdb/hdb_samba4.so', realname='hdb_samba4.so', install_path='${MODULESDIR}/hdb', @@ -67,7 +66,6 @@ bld.SAMBA_LIBRARY('HDB_SAMBA4_PLUGIN', bld.SAMBA_SUBSYSTEM('KDC-SERVER', source='kdc-server.c kdc-proxy.c', - includes=kdc_include, deps=''' krb5samba ldb @@ -83,7 +81,6 @@ elif bld.CONFIG_GET('SAMBA_USES_MITKDC'): bld.SAMBA_SUBSYSTEM('KPASSWD-SERVICE', source=kpasswd_flavor_src, - includes=kdc_include, deps=''' krb5samba samba_server_gensec @@ -106,13 +103,11 @@ bld.SAMBA_SUBSYSTEM('WDC_SAMBA4', bld.SAMBA_SUBSYSTEM('sdb', source='sdb.c', - includes=kdc_include, deps='talloc krb5', ) bld.SAMBA_SUBSYSTEM('sdb_hdb', source='sdb_to_hdb.c', - includes=kdc_include, deps='talloc sdb hdb', autoproto='sdb_hdb.h', enabled=bld.CONFIG_SET('SAMBA4_USES_HEIMDAL') @@ -120,7 +115,6 @@ bld.SAMBA_SUBSYSTEM('sdb_hdb', bld.SAMBA_SUBSYSTEM('sdb_kdb', source='sdb_to_kdb.c', - includes=kdc_include, deps='sdb kdb5', autoproto='sdb_kdb.h', enabled=bld.CONFIG_SET('HAVE_KDB_H') @@ -128,7 +122,6 @@ bld.SAMBA_SUBSYSTEM('sdb_kdb', bld.SAMBA_SUBSYSTEM('PAC_GLUE', source='pac-glue.c', - includes=kdc_include, deps='ldb auth4_sam common_auth samba-credentials samba-hostconfig com_err' ) @@ -143,12 +136,10 @@ bld.SAMBA_LIBRARY('db-glue', source='db-glue.c', deps='ldb auth4_sam common_auth samba-credentials sdb samba-hostconfig com_err RPC_NDR_IRPC MESSAGING', private_library=True, - includes=kdc_include, ) bld.SAMBA_SUBSYSTEM('KPASSWD_GLUE', source='kpasswd_glue.c', - includes=kdc_include, deps='ldb com_err') bld.SAMBA_SUBSYSTEM('MIT_KDC_IRPC', -- Samba Shared Repository