MailScanner --lint errors

[Dimitri Yioulos]

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

spamassassin --lint retuns no errors.

Dimitri

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