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

[Dieter Braun <di...@onevision.com>]

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
>
>

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

["Kevin A. McGrail" <KM...@PCCC.com>]

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