https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8227

Sidney Markowitz <sid...@sidney.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |---
             Status|RESOLVED                    |REOPENED

--- Comment #3 from Sidney Markowitz <sid...@sidney.com> ---
(In reply to Giovanni Bechis from comment #2)
> The bits added to DependencyInfo.pm breaks install on some networks where
> SpamAssassin would work by adding "dns_server 127.0.0.1" option.

Oh, that's a problem for the fix I did for bug 8225 too, because the calls to
Net::DNS::Resolver aren't using the dns_server conf options.

I also didn't think about t/config.dist default is run_net_tests=n which this
check ignores, breaking the install on systems that will not get test failures.

Ok, I'll have to revert that part of the commit. The new t/dnsplatform.t test
is still useful, though. If it fails it signals that the failures in the other
network tests that fail can be ignored, and that dns_server config options may
be necessary to run on that system.

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

Reply via email to