From 4fb2f535ca73dd16738ce4a3b692931fb26227aa Mon Sep 17 00:00:00 2001 From: Petr Spacek Date: Nov 09 2016 12:08:32 +0000 Subject: Build: fix distribution of daemons/ipa-slapi-plugins/ipa-cldap files All the headers are now listed in _SOURCES variable. It seems weird but this is what GNU Automake manual suggests in section 9.2 Header files: Headers used by programs or convenience libraries are not installed. The noinst_HEADERS variable can be used for such headers. However when the header actually belongs to a single convenience library or program, we recommend listing it in the program’s or library’s _SOURCES variable (see Program Sources) instead of in noinst_HEADERS. This is clearer for the Makefile.am reader. noinst_HEADERS would be the right variable to use in a directory containing only headers and no associated library or program. https://fedorahosted.org/freeipa/ticket/6418 Reviewed-By: Martin Basti Reviewed-By: Christian Heimes --- diff --git a/daemons/ipa-slapi-plugins/ipa-cldap/Makefile.am b/daemons/ipa-slapi-plugins/ipa-cldap/Makefile.am index 453f5ee..037c0fc 100644 --- a/daemons/ipa-slapi-plugins/ipa-cldap/Makefile.am +++ b/daemons/ipa-slapi-plugins/ipa-cldap/Makefile.am @@ -24,6 +24,7 @@ libipa_cldap_la_SOURCES = \ ipa_cldap_netlogon.c \ ipa_cldap_worker.c \ ipa_cldap.c \ + ipa_cldap.h \ $(NULL) libipa_cldap_la_LDFLAGS = -avoid-version