Bug 7615 - Minimum Required Perl for SA 4.0.0
Summary: Minimum Required Perl for SA 4.0.0
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamassassin (show other bugs)
Version: 4.0.0
Hardware: PC Windows NT
: P2 blocker
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-05 16:47 UTC by Kevin A. McGrail
Modified: 2018-09-06 12:10 UTC (History)
2 users (show)


Attachment Type Modified Status Actions Submitter/CLA Status
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin A. McGrail 2018-09-05 16:47:08 UTC 
Surfacing this from other tickets:

For 4.X, we need good, stable UTF support which in my experience means using 5.14+.  

RHEL/CentOS 6 ships with 5.10.1 and is EOL in 2020. And RHEL/CentOS 7 ships with 5.16.3.  

Since RHEL won't backport, I suggest we require 5.14.8+ for 4.X especially since I'm expecting distros won't include 4.X except on the next major release.

To make things easier for those with older perls, we can document and even provide some automation to use the system/distro perl to bootstrap a newer version of perl dedicated for SA with something like perlbrew.  It will require compilation tools/libraries but effectively it's like installing your own JRE for a specific product.

See https://github.com/gugod/App-perlbrew under the Synopsis to see just how easily people can add alternate perl versions to their system.  

Any arguments against changing trunk INSTALL to require 5.14.8 as well as the PACKAGING, Makefile.PL and UPGRADE files?  I can also look at making Makefile.PL bootstrap with system perl and download perlbrew to make a newer perl available.
Comment 1 Giovanni Bechis 2018-09-05 19:41:39 UTC 
ok with minimum perl requirements for SA 4.x,
anyway I do not like Makefiles that downloads
"random" packages from internet; I would prefer 
a message like "missing minimum perl requirements,
read UPGRADE file for more informations on how
to install a newer perl via perlbrew" or
something like that.
Comment 2 Kevin A. McGrail 2018-09-06 12:10:47 UTC 
Committed changes that require 5.14 for wrap() in Util.pm and added a test for 5.14.0 to Makefile.

Trunk only: Committed revision 1840214.