On Mon, 1 Dec 2014, Kevin A. McGrail wrote:

On 12/1/2014 8:03 PM, John Hardin wrote:
 On Mon, 1 Dec 2014, Kevin A. McGrail wrote:

>  On 12/1/2014 6:06 PM, John Hardin wrote:
> > > > It looks like as long as we support perl < 5.10.0 then the only clean
> >   solution is can(Mail::SpamAssassin::Conf::perl_min_version_5010000)
> > With perl versions so low in so many distros, I think we have to > implement the perl_min_version function. Do you want me to take a stab > at it?

 Do we want to be able to check every major perl version (i.e., define
 perl_min_version_5010000, perl_min_version_5011000,
 perl_min_version_5012000, perl_min_version_5013000,
 perl_min_version_5014000, perl_min_version_5015000,
 perl_min_version_5016000, etc.) ?

For now, the only issue that has ever arisen in years is the 5010000 test so I would stick with just that for now.

Ok.

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=7107

--
 John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
 jhar...@impsec.org    FALaholic #11174     pgpk -a jhar...@impsec.org
 key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
  Our government should bear in mind the fact that the American
  Revolution was touched off by the then-current government
  attempting to confiscate firearms from the people.
-----------------------------------------------------------------------
 14 days until Bill of Rights day

Reply via email to