# New Ticket Created by Andy Dougherty
# Please include the string: [perl #16965]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt2/Ticket/Display.html?id=16965 >
This patch gathers together different testing dependencies into a new
test_prep target, and then uses that target where appropriate. This
should reclaim the Solaris 32-bit JIT tinderbox -- it was failing
because the testj: target didn't have the proper libparrot.a dependency.
Whatever the final resolution of that dependency, this patch consolidates
the debate into one test_prep target.
diff -r -u parrot-orig/config/gen/makefiles/root.in
parrot-andy/config/gen/makefiles/root.in
--- parrot-orig/config/gen/makefiles/root.in Tue Sep 3 10:28:29 2002
+++ parrot-andy/config/gen/makefiles/root.in Tue Sep 3 10:46:00 2002
@@ -447,22 +447,25 @@
#
###############################################################################
-test : $(TEST_PROG) ${blib_lib_libparrot_a} assemble.pl test_dummy
+test : $(TEST_PROG) test_prep test_dummy
+
+# Common prep for all test targets
+test_prep : ${blib_lib_libparrot_a} assemble.pl
test_dummy :
$(PERL) t/harness $(TEST_PROG_ARGS)
-testp : $(TEST_PROG) assemble.pl blib/lib/libcore_prederef$(SO)
blib/lib/libparrot$(SO) $(TEST_PROG_SO) test_dummy_p
+testp : $(TEST_PROG) test_prep blib/lib/libcore_prederef$(SO) blib/lib/libparrot$(SO)
+$(TEST_PROG_SO) test_dummy_p
test_dummy_p :
$(PERL) t/harness $(TEST_PROG_ARGS) -P
-testj : $(TEST_PROG) assemble.pl test_dummy_j
+testj : $(TEST_PROG) test_prep test_dummy_j
test_dummy_j :
$(PERL) t/harness $(TEST_PROG_ARGS) -j
-quicktest : $(TEST_PROG) assemble.pl quicktest_dummy
+quicktest : $(TEST_PROG) test_prep quicktest_dummy
quicktest_dummy :
$(PERL) t/harness $(TEST_PROG_ARGS) quick
--
Andy Dougherty [EMAIL PROTECTED]
Dept. of Physics
Lafayette College, Easton PA 18042
