Re: Argument "perl_version" isn't numeric

[jdow]

On 2014-11-30 14:29, John Hardin wrote:
On Sun, 30 Nov 2014, jdow wrote:
 On Sun, 30 Nov 2014, Benny Pedersen wrote:
>  On 30. nov. 2014 22.15.12 John Hardin <jhar...@impsec.org> wrote:
> > >       if version > 3.004001
> >         if perl_version >= 5.010000
> >           body NON_588_COMPATIBLE_RE_SYNTAX /\w++/
> >         endif
> >       endif
> >  If this works now in spamassassin 3.3.2, problem solved, can i send >
bitcoins
>  somewhere ? :)

 It "works" in 3.3.2 *IF* you accept the warning about perl_version
 being non-numeric that this entire thread is about. The
 non-5.8.8-compatible RE is skipped, because the perl_version
 conditional fails to false on the type warning and doesn't include the
 rule.

Would a corrected syntax version of this work?

if version > 3.004001 && perl_version >= 5.010000
  body NON_588_COMPATIBLE_RE_SYNTAX /\w++/
endif

Yes. That *does* work.

Thank you! I think you just solved it.

Here is another idea that might be helpful if....

    if ((ref( perl_version ) == SCALAR ) && (perl_version >= 5.010000))
        body yatta yatta
    endif

Simply determine that the data type for perl_version will work when evaluating 
the second half if the if.

{^_^}
{^_^}