You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by jm...@apache.org on 2007/02/20 14:35:10 UTC

svn commit: r509555 - /spamassassin/trunk/build/automc/run_preflight

Author: jm
Date: Tue Feb 20 05:35:10 2007
New Revision: 509555

URL: http://svn.apache.org/viewvc?view=rev&rev=509555
Log:
should run preflight mass-checks anyway even if lint checks fail for that set of rules files

Modified:
    spamassassin/trunk/build/automc/run_preflight

Modified: spamassassin/trunk/build/automc/run_preflight
URL: http://svn.apache.org/viewvc/spamassassin/trunk/build/automc/run_preflight?view=diff&rev=509555&r1=509554&r2=509555
==============================================================================
--- spamassassin/trunk/build/automc/run_preflight (original)
+++ spamassassin/trunk/build/automc/run_preflight Tue Feb 20 05:35:10 2007
@@ -61,11 +61,18 @@
 # lint those rule files, and abort if they fail; this is a sanity check
 # now that we have mailed-in rules used in this mass-check
 #
+# Feb 20 2007 jm: actually, go ahead and mass-check anyway.  Note that
+# the "make test" continuous integration buildbot will issue warnings
+# about the lint failure, so we don't need to do that here.
+#
 if (run ("../spamassassin --lint -x -C tstrules", 1) != 0) {
-  warn "*** lint failed; exiting without mass-check! ***\n\n";
-  warn "Deleting 'mailed.cf', if it exists.\n"; unlink $BYMAILCF;
-  warn "Rule files:\n"; system ("cat tstrules/*.cf");
-  die "lint failure";
+  warn "*** lint failed; performing mass-check ANYWAY! ***\n\n";
+
+  # still, delete the mailed-in stuff.
+  warn "Deleting 'mailed.cf', if it exists.\n";
+  unlink $BYMAILCF;
+  unlink "tstrules/70_mailed.cf";
+
 } else {
   print "lint passed.\n";
 }