AUDIT: r21570 - gnucash/trunk - Fix clean targets and DIST_SOURCES

[John Ralls]

Author: jralls
Date: 2011-11-17 16:27:40 -0500 (Thu, 17 Nov 2011)
New Revision: 21570
Trac: http://svn.gnucash.org/trac/changeset/21570

Modified:
   gnucash/trunk/configure.ac
   gnucash/trunk/lib/Makefile.am
   gnucash/trunk/src/Makefile.am
   gnucash/trunk/src/app-utils/Makefile.am
   gnucash/trunk/src/backend/Makefile.am
   gnucash/trunk/src/business/business-core/Makefile.am
   gnucash/trunk/src/business/business-gnome/Makefile.am
   gnucash/trunk/src/business/dialog-tax-table/Makefile.am
   gnucash/trunk/src/core-utils/Makefile.am
   gnucash/trunk/src/engine/Makefile.am
   gnucash/trunk/src/gnc-module/Makefile.am
   gnucash/trunk/src/gnome-utils/Makefile.am
   gnucash/trunk/src/import-export/Makefile.am
   gnucash/trunk/src/optional/Makefile.am
   gnucash/trunk/src/optional/python-bindings/tests/Makefile.am
   gnucash/trunk/src/python/Makefile.am
   gnucash/trunk/src/report/Makefile.am
   gnucash/trunk/src/report/report-gnome/Makefile.am
   gnucash/trunk/src/report/report-system/Makefile.am
Log:
Fix clean targets and DIST_SOURCES

Several subdirectories have a subdirectory "gnucash", which clean tried to remove by including in the CLEANFILES variable in Makefile.am. This doesn't work, because directories can't be removed with rm, they require rm -r. Add a clean-local: target which calls rm -r on the gnucash subdirectory.

Some Makefile.ams used an AC_SUBSTed variable to define an optionally-compiled subdirectory, then included the variable in SUBDIRS and the actual directory name in DIST_SUBDIRS. This breaks the distclean and maintainer-clean targets when the optional directory is built, because it is then processed twice, and there is no Makefile the second time so make halts. Replaced the AC_SUBSTed variables with ones local to the Makefile.am, switched with an AM_CONDITIONAL. That way automake can figure out the dist_subdirs on its own.

BP