LP#1791184: ensure that 'make clean' doesn't delete pingest.pl
authorGalen Charlton <gmc@equinoxinitiative.org>
Thu, 6 Sep 2018 22:57:05 +0000 (18:57 -0400)
committerJason Stephenson <jason@sigio.com>
Thu, 6 Sep 2018 22:55:40 +0000 (18:55 -0400)
This patch corrects an issue where pingest.pl was treated as if
it were generated from a pingest.pl.in source file during the configure
step, meaning that it got deleted by 'make clean'.  This patch also
adds a couple helpful comments.

To test
-------
[1] Apply the patch.
[2] From a git checkout, run the configure and make steps, followed
    by a 'make clean'. Verify that pingest.pl is not deleted from
    the source tree.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/src/Makefile.am

index 444846b..a9de4a9 100644 (file)
@@ -75,7 +75,8 @@ core_scripts =   $(examples)/oils_ctl.sh \
                 $(supportscr)/sitemap_generator \
                 $(srcdir)/extras/eg_config \
                 $(srcdir)/extras/openurl_map.pl \
-                $(srcdir)/extras/import/marc_add_ids
+                $(srcdir)/extras/import/marc_add_ids \
+                $(supportscr)/pingest.pl
 
 installautojs = $(autojsbinscripts)
 
@@ -118,6 +119,8 @@ if BUILDEGJAVA
 OILSJAVA_DIR = java
 endif
 
+# scripts that are generated from *.in files and can
+# be deleted by a make clean
 gen_scripts = \
        @srcdir@/extras/fast-extract \
        @srcdir@/extras/import/marc2are.pl \
@@ -129,9 +132,10 @@ gen_scripts = \
        $(supportscr)/authority_authority_linker.pl \
        $(supportscr)/eg_db_config \
        $(supportscr)/marc_export \
-       $(supportscr)/offline-blocked-list.pl \
-       $(supportscr)/pingest.pl
+       $(supportscr)/offline-blocked-list.pl
 
+# config files that are generated from *.in files and can
+# be deleted by a make clean
 gen_docs = \
        $(examples)/apache/eg.conf \
        $(examples)/apache/eg_vhost.conf \