IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2024-12-24 16:10:02 -06:00
Code Issues Packages Projects Releases Wiki Activity

Build: fix distribution of daemons/ipa-slapi-plugins/ipa-exdom-extop files

Browse Source 
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 <mbasti@redhat.com>
Reviewed-By: Christian Heimes <cheimes@redhat.com>
This commit is contained in:
Petr Spacek 2016-11-02 19:42:35 +01:00 committed by Martin Babinsky
parent 4fb2f535ca
commit 14bce67cf0
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
daemons/ipa-slapi-plugins/ipa-extdom-extop/Makefile.am
View File

@ -21,6 +21,7 @@ plugin_LTLIBRARIES = \
$(NULL) $(NULL)
libipa_extdom_extop_la_SOURCES = \ libipa_extdom_extop_la_SOURCES = \
ipa_extdom.h \
ipa_extdom_extop.c \ ipa_extdom_extop.c \
ipa_extdom_common.c \ ipa_extdom_common.c \
$(NULL) $(NULL)
@ -66,6 +67,7 @@ app_DATA = \
EXTRA_DIST = \ EXTRA_DIST = \
README \ README \
test_data \
$(app_DATA) \ $(app_DATA) \
$(NULL) $(NULL)