gnuastro-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[gnuastro-commits] master a335d1f: MakeProfile's radeccat test input in


From: Mohammad Akhlaghi
Subject: [gnuastro-commits] master a335d1f: MakeProfile's radeccat test input in tarball
Date: Wed, 10 Aug 2016 21:48:01 +0000 (UTC)

branch: master
commit a335d1fdefcbf93d3dad743ae89df16d6c9b2c44
Author: Mohammad Akhlaghi <address@hidden>
Commit: Mohammad Akhlaghi <address@hidden>

    MakeProfile's radeccat test input in tarball
    
    In commit f5a1bfc, the ability to read the profile positions in the world
    coordinate system was included and a test was written for it. However, I
    forgot to run `make distcheck' before making the commit (and pushing to the
    main repository). After running `make distcheck', I noticed that the test's
    input catalog has not been included in the distribution files, so it will
    be missing in tar-ball installs causing the test to fail. With this commit
    the issue is solved.
    
    While modifying the file, I also noticed that the `tests/during-dev.sh'
    script (from commit cb7d44a) was mistakenly included in the distribution
    tarball from the top-level `Makefile.am'. This script is now also managed
    with the `tests/Makefile.am' now.
    
    Also one part of the `tests/during-dev.sh' comments were made more clear
    and corrected and a paragraph was added to the comments.
---
 Makefile.am         |    3 +--
 tests/Makefile.am   |    6 +++---
 tests/during-dev.sh |   14 ++++++++++----
 3 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index 5a7cf75..aeab85b 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -117,8 +117,7 @@ EXTRA_DIST = bootstrap bootstrap.conf genauthors 
.dir-locals.el .version   \
              tmpfs-config-make bootstrapped/m4/gnulib-cache.m4             \
              bootstrapped/README .autom4te.cfg lib/gnuastro/commonargs.h   \
              lib/gnuastro/commonparams.h lib/gnuastro/fixedstringmacros.h  \
-             lib/gnuastro/neighbors.h lib/gnuastro/README                  \
-             tests/during-dev.sh
+             lib/gnuastro/neighbors.h lib/gnuastro/README
 
 
 
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 8720303..d188530 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -43,9 +43,9 @@ header/delete.sh imgstat/basicstats.sh 
subtractsky/subtractsky.sh           \
 noisechisel/noisechisel.sh mkcatalog/simple.sh arithmetic/snimage.sh        \
 arithmetic/onlynumbers.sh cosmiccal/simpletest.sh
 
-EXTRA_DIST = $(TESTS) mkprof/mkprofcat1.txt                    \
-mkprof/mkprofcat1_mask.txt mkprof/mkprofcat2.txt               \
-mkprof/mkprofcat3.txt mkprof/mkprofcat4.txt imgcrop/cat.txt
+EXTRA_DIST = $(TESTS) during-dev.sh mkprof/mkprofcat1.txt               \
+mkprof/mkprofcat1_mask.txt mkprof/mkprofcat2.txt mkprof/mkprofcat3.txt \
+mkprof/mkprofcat4.txt mkprof/radeccat.txt imgcrop/cat.txt
 
 CLEANFILES = *.log *.txt *.jpg *.fits *.pdf *.eps
 
diff --git a/tests/during-dev.sh b/tests/during-dev.sh
index 2dc4935..1195c89 100755
--- a/tests/during-dev.sh
+++ b/tests/during-dev.sh
@@ -25,13 +25,19 @@
 # manually before each major development activity.
 #
 # This file will be changed alot during development. So please make sure
-# you have left the variable values empty before making commit so each
-# developer is guided to set them for their own case. Alternatively you can
-# reset this file to its version controlled status before making your
-# commit:
+# you have left the variable values empty before committing. This will
+# remind each developer running this script to set the values based on
+# their particular work. Alternatively you can reset this file to its
+# version controlled status before committing your main work with the
+# command below:
 #
 #     git checkout -- tests/during-dev.sh
 #
+# This file can also be used as a model to write a test for the work you
+# have done (to be checked with `make check'). Just copy and paste an
+# existing test from the utility and replace the last few lines based on
+# this file.
+#
 # Original author:
 #     Mohammad Akhlaghi <address@hidden>
 # Contributing author(s):



reply via email to

[Prev in Thread] Current Thread [Next in Thread]