MailScanner --lint errors

[Julian Field]

Dimitri Yioulos wrote:
> On Monday 07 April 2008 3:00 pm, Julian Field wrote:
>   
>> Dimitri Yioulos wrote:
>>     
>>> On Monday 07 April 2008 1:18 pm, Julian Field wrote:
>>>       
>>>> Dimitri Yioulos wrote:
>>>>         
>>>>> On Monday 07 April 2008 10:31 am, Kai Schaetzl wrote:
>>>>>           
>>>>>> Dimitri Yioulos wrote on Mon, 7 Apr 2008 07:53:56 -0400:
>>>>>>             
>>>>>>> I didn't get that with mailscanner-4.65.3-1, my last version before
>>>>>>> upgrading to the latest.
>>>>>>>               
>>>>>> sounds rather like a problem with SA. Did you also upgrade SA? You may
>>>>>> be missing a now required Perl module. A timing package?
>>>>>>
>>>>>> Kai
>>>>>>
>>>>>> --
>>>>>> Kai Schätzl, Berlin, Germany
>>>>>> Get your web at Conactive Internet Services: http://www.conactive.com
>>>>>>             
>>>>> Kai,
>>>>>
>>>>> I'm running the latest (RPM) version of SA - 3.2.4.  All of the perl SA
>>>>> modules look to be up-to-date, too.
>>>>>
>>>>> I took a look at
>>>>> /usr/lib/perl5/vendor_perl/5.8.0/Mail/SpqamAssassin/Dns.pm line 371,
>>>>> which read "$total_waiting_time += $waiting_time".  Just for fun, I
>>>>> deleted the +, ran MS --lint, and the first error was gone.  However,
>>>>> I'm not sure if the "+=" isn't a valid construct, and what the
>>>>> consequences of my change would be (and so may just put back the +).
>>>>>           
>>>> You can't just remove "+" signs like that, sorry!
>>>> That line is a shorthand for
>>>> $total_waiting_time = $total_waiting_time + $waiting_time;
>>>>
>>>>         
>>>>> As to the second error, I see the line in Dns.pm, but have no idea what
>>>>> it does.  Googling has turned up little.
>>>>>
>>>>> Dimitri
>>>>>           
>>>> Jules
>>>>
>>>> --
>>>> Julian Field MEng CITP CEng
>>>> www.MailScanner.info
>>>> Buy the MailScanner book at www.MailScanner.info/store
>>>>         
>>> Thanks, Jules.  I figured as much.  So that's what "+=" means.  Can these
>>> errors be ignored as being tivial?
>>>       
>> Depends what the errors were. Your previous message didn't include the
>> error messages.
>>
>> Jules
>>
>> --
>> Julian Field MEng CITP CEng
>> www.MailScanner.info
>> Buy the MailScanner book at www.MailScanner.info/store
>>
>>     
>
> Sorry, they were edited out by a previous poster.  MailScanner --lint returns:
>
> Checking for SpamAssassin errors (if you use it)...
> SpamAssassin temporary working directory 
> is /var/spool/MailScanner/incoming/SpamAssassin-Temp
> SpamAssassin temp dir = /var/spool/MailScanner/incoming/SpamAssassin-Temp
> Using SpamAssassin results cache
> Connected to SpamAssassin cache database
> Use of uninitialized value in addition (+) 
> at /usr/lib/perl5/vendor_perl/5.8.0/Mail/SpamAssassin/Dns.pm line 371.
>   
That's pretty irrelevant, just a minor warning.
> plugin: eval failed: Can't locate object method "log_lookups_timing" via 
> package "Mail::SpamAssassin::AsyncLoop" 
> at /usr/lib/perl5/vendor_perl/5.8.0/Mail/SpamAssassin/Dns.pm line 381.
> SpamAssassin reported no errors.
>   
This appears to imply a problem in the DNS lookups done by SpamAssassin. 
Do the DNS lookups done by SpamAssassin still appear to work okay? If 
so, ignore it. If not, then I would take that problem to the 
SpamAssassin mailing list, once you have made sure you are running the 
latest version of SpamAssassin ("MailScanner -v" will tell you what 
version it is).

> spamassassin --lint retuns no errors.
>   
What about "spamassassin --lint -D"? Does its output show any warnings 
about DNS lookups?

Jules

-- 
Julian Field MEng CITP CEng
www.MailScanner.info
Buy the MailScanner book at www.MailScanner.info/store

MailScanner customisation, or any advanced system administration help?
Contact me at Jules at Jules.FM

PGP footprint: EE81 D763 3DB0 0BFD E1DC 7222 11F6 5947 1415 B654
PGP public key: http://www.jules.fm/julesfm.asc


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.