[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnuastro-commits] (no subject)
From: |
Mohammad Akhlaghi |
Subject: |
[gnuastro-commits] (no subject) |
Date: |
Mon, 30 May 2016 08:22:00 +0000 (UTC) |
branch: master
commit f8c19a313f6a70a2c687aa736c57966d9f2d600b
Author: Mohammad Akhlaghi <address@hidden>
Date: Mon May 30 17:01:37 2016 +0900
tests/.gnuastro directory is now cleaned
The `tests/.gnuastro' directory is built by the `tests/prepconf.sh' script
to keep all the configuration files for testsing. However, to clean it, we
had relied on Automake's `CLEANFILES' variable. As the name suggests, this
variable is only built for files, not directories. So the build directory
would not get cleaned. Spotting this problem was hard because Automake
doesn't abort in such cases and the notice was too deeply buried in the
other outputs.
This problem was fixed using Automake's `clean-local' rule.
---
tests/Makefile.am | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 80f0711..e29a4b0 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -49,8 +49,13 @@ EXTRA_DIST = $(TESTS) mkprof/mkprofcat1.txt
\
mkprof/mkprofcat1_mask.txt mkprof/mkprofcat2.txt \
mkprof/mkprofcat3.txt mkprof/mkprofcat4.txt imgcrop/cat.txt
-CLEANFILES = *.log *.txt *.jpg *.fits *.pdf *.eps .gnuastro/*.conf \
-.gnuastro
+CLEANFILES = *.log *.txt *.jpg *.fits *.pdf *.eps
+
+# CLEANFILES is only for files, not directories. Therefore we are using
+# Automake's extending rules to clean the temporary `.gnuastro' directory
+# that was built by the `prepconf.sh' scripot. See "Extending Automake
+# rules", and the "What Gets Cleaned" sections of the Automake manual.
+clean-local:; rm -rf .gnuastro