Re: sa-update fails - bug 6702 reappearing? --- solved

[Dieter Braun]

Thanks a lot for this hint - that's been it!

This had happened: The files /etc/mail/spamassassin/*.pre, that already 
existed, were not replaced by the new versions. Only the new file 
v340.pre was copied to /etc/mail/spamassassin.


After moving the existing *.pre files somewhere else and copying the new 
*.pre files, sa-update worked without any problem.


But now I'm wondering, why the *.pre files aren't included in 
/usr/lib/perl5/site_perl/5.8.8/x86_64-linux-thread-multi/auto/Mail/SpamAssassin/.packlist 
(and not included in the file list shown after make uninstall)? But 
that's another topic.


(Sorry, that I couldn't come back to this topic earlier - many 
colleagues are ill and need substitution.)


Best regards and many thanks,
Dieter


Am 04.03.2014 15:09, schrieb Mark Martinec:

Dieter Braun wrote:


# sa-update
rules: failed to run T_HEADER_FROM_DIFFERENT_DOMAINS test, skipping:
 (Can't locate object method check_equal_from_domains via package
   Mail::SpamAssassin::PerMsgStatus at (eval 1008) line 97.


You are missing a line:
  loadplugin Mail::SpamAssassin::Plugin::HeaderEval
in one of your .pre files.

That line is normally in v320.pre. You may have commented it out,
or the file v320.pre is missing from your SpamAssassin configuration
directory.

Btw, the rule T_HEADER_FROM_DIFFERENT_DOMAINS should have
been conditionalized to only apply if the plugin HeaderEval
is available.

  Mark




attachment: dieter_braun.vcf

Re: sa-update fails - bug 6702 reappearing? --- solved

[Kevin A. McGrail]

On 3/5/2014 8:36 AM, Dieter Braun wrote:

Thanks a lot for this hint - that's been it!

This had happened: The files /etc/mail/spamassassin/*.pre, that 
already existed, were not replaced by the new versions. Only the new 
file v340.pre was copied to /etc/mail/spamassassin.


After moving the existing *.pre files somewhere else and copying the 
new *.pre files, sa-update worked without any problem.


But now I'm wondering, why the *.pre files aren't included in 
/usr/lib/perl5/site_perl/5.8.8/x86_64-linux-thread-multi/auto/Mail/SpamAssassin/.packlist 
(and not included in the file list shown after make uninstall)? But 
that's another topic.


User configuration files (.pre files are intended to be changed) are 
usually exempted to avoid removing important local configuration.


The bug really was on our end where we were not testing all the plugins 
needed for that rule and you happened to have one of the two disabled.


You just helped identify the bug.

(Sorry, that I couldn't come back to this topic earlier - many 
colleagues are ill and need substitution.)

Better health to them!

regards,
KAM