http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5599


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.2.4




------- Additional Comments From [EMAIL PROTECTED]  2007-08-16 02:31 -------
Thanks for the patch!  I'm afraid it needs just a little more work, though ;)

- List::Util wasn't added to perl core until perl 5.7.3, which means that it's
  not bundled with one of our supported platforms, perl 5.6.1.  This is a
  problem; adding a new prerequisite module to SpamAssassin is a big deal for
  us, since it makes SpamAssassin harder to install.  If you could redo the
  patch with its own copy of a shuffle() routine, that'd be good.

- is there any reason for this?

         else {
-          dbg("dns: NS lookup of $domain using $ns failed, no results found");
+          warn("dns: NS lookup of $domain using $ns failed, no results found");
           next;

  I'm concerned this will appear for every NXDOMAIN, which could be very
  noisy on a busy server.

- I think we'll need a CLA for this patch:
  http://wiki.apache.org/spamassassin/AboutClas





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to