Ticket #21 (closed defect: fixed)

Opened 2 years ago

Last modified 1 year ago

DBI required even if not used

Reported by: r.berber@cactus-soft.dyndns.org Assigned to: decoder
Priority: major Milestone: Development Release Version 3.5
Component: Configuration Version:
Keywords: DBI Cc:

Description

With 3.5.0-rc1 after installing, configuring with no database (focr_enable_image_hashing is commented out, but the same if "focr_enable_image_hashing 2" is used) the package doesn't pass a lint test:

$ spamassassin --lint [3932] warn: plugin: failed to parse plugin /etc/mail/spamassassin/FuzzyOcr.pm: Can't locate DBI.pm in @INC (@INC contains: .. /etc/mail/spamassassin /usr/lib/perl5/site_perl/5.8/cygwin /usr/lib/perl5/site_perl/5.8 /usr/lib/perl5/5.8/cygwin /usr/lib/perl5/5.8 /usr/lib/perl5/vendor_perl/5.8/cygwin /usr/lib/perl5/vendor_perl/5.8) at /etc/mail/spamassassin/FuzzyOcr/Config.pm line 161. ...

The problem is that Config.pm look unconditionally for DBI, it shouldn't if the configuration doesn't use MySQL.

Attachments

dbi.patch (1.3 kB) - added by decoder on 11.12.2006 00:03:42.
Patch for the DBI issue

Change History

11.12.2006 00:03:42 changed by decoder

  • attachment dbi.patch added.

Patch for the DBI issue

11.12.2006 00:03:53 changed by decoder

  • status changed from new to assigned.

Thank you very much for the report, I fixed Config.pm accordingly, a patch is attached :) Tell me if it fails :)

Best regards,

Chris

13.12.2006 21:18:16 changed by decoder

reopened, the bug persists. The use calls are redundant, will fix in next patchset.

14.12.2006 00:24:40 changed by decoder

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed in SVN now finally (hopefully). Patchset 3 coming soon.

11.04.2007 15:00:17 changed by Emily

15.08.2007 02:25:45 changed by anonymous


Add/Change #21 (DBI required even if not used)




Action