You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by fe...@apache.org on 2004/08/01 02:18:25 UTC

svn commit: rev 31067 - in spamassassin/trunk: lib/Mail spamd

Author: felicity
Date: Sat Jul 31 17:18:24 2004
New Revision: 31067

Modified:
   spamassassin/trunk/lib/Mail/SpamAssassin.pm
   spamassassin/trunk/spamd/spamd.raw
Log:
bug 3651: if calling compile_now(), the available Bayes DB will remain tied.  a warning will then pop up (sanity_check_untie) before untieing.  so explicitly untie the DB when we're finished.

Modified: spamassassin/trunk/lib/Mail/SpamAssassin.pm
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin.pm	(original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin.pm	Sat Jul 31 17:18:24 2004
@@ -1130,6 +1130,7 @@
   $self->{conf}->{use_bayes_rules} = $use_bayes_rules_value;
   $status->finish();
   $mail->finish();
+  $self->finish_learner();
 
   # load SQL modules now as well
   my $dsn = $self->{conf}->{user_scores_dsn};

Modified: spamassassin/trunk/spamd/spamd.raw
==============================================================================
--- spamassassin/trunk/spamd/spamd.raw	(original)
+++ spamassassin/trunk/spamd/spamd.raw	Sat Jul 31 17:18:24 2004
@@ -737,7 +737,7 @@
 # process are written with the right ownership and everything.
 preload_modules_with_tmp_homedir();
 
-# bayes DBs will be tied() at this point, so untie them and such.
+# bayes DBs may still be tied() at this point, so untie them and such.
 $spamtest->finish_learner();
 
 # If we're going to be switching users in check(), let's backup the