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

            Bug ID: 7143
           Summary: Evaluate makemaker version using perl best practices
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Hardware: PC
                OS: Windows 7
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Building & Packaging
          Assignee: [email protected]
          Reporter: [email protected]

As per:

http://perldoc.perl.org/perlmodstyle.html#Version-numbering

Makefile.PL can be simplified with:

--- Makefile.PL.orig    2015-02-20 14:09:08.821327624 -0600
+++ Makefile.PL 2015-02-20 14:09:42.725326495 -0600
@@ -8,12 +8,9 @@
 use ExtUtils::MakeMaker;

 # Store the version for later use
-my $mm_version = $ExtUtils::MakeMaker::VERSION;
-
 # avoid stupid 'Argument "6.30_01" isn't numeric in numeric ge (>=)' warnings;
 # strip off the beta subversion noise that causes the trouble.
-$mm_version =~ s/_\S+$//;      # "6.30_01" => "6.30"
-$mm_version += 0;
+my $mm_version = eval $ExtUtils::MakeMaker::VERSION;

 if ($mm_version < 6.17) {
   die "SpamAssassin Makefile.PL requires at least ExtUtils::MakeMaker v6.17";

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

Reply via email to