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

            Bug ID: 8195
           Summary: Optional modules in tests are considered required by
                    CPAN
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Hardware: All
                OS: All
            Status: NEW
          Severity: minor
          Priority: P2
         Component: Building & Packaging
          Assignee: dev@spamassassin.apache.org
          Reporter: sid...@sidney.com
  Target Milestone: Undefined

Devel::Cycle and Text::Diff are listed in Makefile.PL as TEST_REQUIRES even
though the tests that use them are written to skip without failing if the
module is not installed. perl Makefile.PL does not distinguish between REQUIRES
and TEST_REQUIRES, so trying to build on a system without, for example
Devel::Cycle produces a misleading warning

  Warning: prerequisite Devel::Cycle 0 not found.

We put the optional modules in the recommends list in Makefile.PL and in an
OPTIONAL_MODULES list in lib/Mail/SpamAssassin/Util/DependencyInfo.pm

That's the right thing to do with any modules tghat are optionally used by
tests. DependencyInfo has description strings in the list where we can explain
that the missing module is optional and is only used for some tests that will
be skipped.

I'll check the tests to look for other modules that are optionally loaded and
add them to the lists too if I find any.

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

Reply via email to