bind-dyndb-ldap

#226 Detect and propagate atomic libraries like bind9
Merged by abbra. Opened by pemensik.
pemensik/bind-dyndb-ldap bind-9.18-support  into  master

BIND9 headers expect atomic definitions are configured before they are
included. It needs adding atomic libraries detection in configure AND
including config.h before any ISC headers are included.

Move dyndb-config.h before isc headers anywhere where needed.

This change is needed to build with bind-9.18.19, bug https://bugzilla.redhat.com/show_bug.cgi?id=2232346.

LGTM, thank you!

Pull-Request has been merged by abbra

wait, this seems incomplete. At least failed my copr build: https://copr.fedorainfracloud.org/coprs/pemensik/bind/build/6440731/

 
checking libdns version... 1819
./configure: line 14379: syntax error near unexpected token `atomic'
./configure: line 14379: `   AX_SAVE_FLAGS(atomic)'

It looks like this change depends on Autoconf archive which we didn't use before. So BuildRequires: autoconf-archive is missing

Correct. bind9 bundles those definitions, but I guess if autoconf-archive fixes it, it should be preferred anyway.

At least, both Fedora and RHEL 8+ do have autoconf-archive package so we can rely on it.
Metadata
None
No Tags
Changes Summary 18
+72 -0
file changed
configure.ac
+2 -1
file changed
src/empty_zones.c
+2 -0
file changed
src/fs.c
+2 -1
file changed
src/fwd_register.c
+1 -0
file changed
src/krb5_helper.c
+2 -1
file changed
src/ldap_convert.c
+3 -1
file changed
src/ldap_entry.c
+2 -0
file changed
src/lock.c
+2 -0
file changed
src/log.c
+2 -0
file changed
src/metadb.c
+3 -2
file changed
src/mldap.c
+2 -1
file changed
src/rbt_helper.c
+2 -0
file changed
src/semaphore.c
+2 -1
file changed
src/settings.c
+2 -0
file changed
src/str.c
+2 -1
file changed
src/syncptr.c
+2 -0
file changed
src/zone.c
+2 -1
file changed
src/zone_register.c
Powered by Pagure 1.0.0
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.