[Freeipa-devel] [freeipa PR#251][comment] Add rebuild rule for template files
URL: https://github.com/freeipa/freeipa/pull/251 Title: #251: Add rebuild rule for template files pspacek commented: """ I'm going to provide a proper fix as agreed on meeting today. """ See the full comment at https://github.com/freeipa/freeipa/pull/251#issuecomment-261987023 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
[Freeipa-devel] [freeipa PR#251][comment] Add rebuild rule for template files
URL: https://github.com/freeipa/freeipa/pull/251 Title: #251: Add rebuild rule for template files tiran commented: """ I created a ticket for you, https://fedorahosted.org/freeipa/ticket/6498 . This **exact (!)** AC_SUBST works for me. Additional newlines cause CONFIG_STATUS_DEPENDENCIES to be an empty rule. ``` # re-run configure after templates have been altered AC_SUBST([CONFIG_STATUS_DEPENDENCIES], ['$(top_srcdir)/daemons/ipa-version.h.in $(top_srcdir)/freeipa.spec.in $(top_srcdir)/ipapython/version.py.in $(top_srcdir)/ipasetup.py.in $(top_srcdir)/install/ui/src/libs/loader.js.in']) ``` """ See the full comment at https://github.com/freeipa/freeipa/pull/251#issuecomment-261485319 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
[Freeipa-devel] [freeipa PR#251][comment] Add rebuild rule for template files
URL: https://github.com/freeipa/freeipa/pull/251 Title: #251: Add rebuild rule for template files pspacek commented: """ NACK: - the variable is defined incorrectly, it should be in configure.ac (see Automake manual chapter [16 Rebuilding Makefiles](https://www.gnu.org/software/automake/manual/html_node/Rebuilding.html)) - file `install/ui/src/libs/loader.js` is missing in the list - please reference build system refactoring ticket in the commit message - even more importantly, there is a conceptual problem: I'm not even sure that this is the right approach. CONFIG_STATUS_DEPEDENCIES is intended for re-building Makefiles and related infrastructure, not individual files generated from templates. With this patch it will re-run configure and config.status for the whole tree. The correct way to do this is to re-run config.status only for particular file. (config.status is the script which is doing actual variable substitution into templates.) Fixing this might require moving these particular template substitutions from `AC_CONFIG_FILES` to Makefile.am in particular directories. """ See the full comment at https://github.com/freeipa/freeipa/pull/251#issuecomment-261470338 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code