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";
}