You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2014/12/02 03:09:16 UTC

[Bug 7107] RFE: if() preprocessor directive should support a test for perl version

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

--- Comment #10 from John Hardin <jh...@impsec.org> ---
Created attachment 5255
  --> https://issues.apache.org/SpamAssassin/attachment.cgi?id=5255&action=edit
Add Mail::SpamAssassin::Conf::perl_min_version_5010000()

Okay, perl_version is an attractive solution, but the warning it generates on
older SA causes administrative heartburn.

That can be avoided by doing this:

if version > 3.004.001 && perl_version >= 5.010000
 ...
endif

UNLESS the perl version is < 5.10, in which case that STILL generates a type
warning.

So:

Patch adds Mail::SpamAssassin::Conf::perl_min_version_5010000() so that we can
do this:

if can(Mail::SpamAssassin::Conf::perl_min_version_5010000)
 ...
endif

Patch includes documentation, but it seems clumsy to me. Comments solicited.
Simply not mention perl_version at all? Revert it completely?

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