On 2014-12-02 10:10, John Hardin wrote:
On Tue, 2 Dec 2014, Burnie wrote:

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

>  On 12/1/2014 8:03 PM, John Hardin wrote:
> >  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

Just FYI: The nested if example in the patch/doc will still
give a lint warning for perl < 5.10

  if can(Mail::SpamAssassin::Conf::perl_min_version_5010000)
    if version > 3.004001 && perl_version >= 5.018000
     body  INVALID_RE_SYNTAX_IN_PERL_BEFORE_5_18  /(?[ \p{Thai} & \p{Digit} ])/
    endif
  endif

Dec  2 03:53:48.550 [30251] warn: Argument "perl_version" isn't numeric in
numeric ge (>=) at (eval 2521) line 2.

ARGH!

Well, I suppose we're back to hoping the distro maintainers accept the
perl_version patch for their LTR release versions of older SA releases.

- IMHO, that single '+' character may be the single most annoying character in
SA for years? :-\

indeed.

Does this show the error?

   if can(Mail::SpamAssassin::Conf::perl_min_version_5010000)
     && version > 3.004001 && perl_version >= 5.018000
      body  INVALID_RE_SYNTAX_IN_PERL_BEFORE_5_18  /(?[ \p{Thai} & \p{Digit} ])/
   endif

Perhaps the same trick can (almost) work again.

{^_^}

Reply via email to