Re: Argument "perl_version" isn't numeric

[Reindl Harald]

Am 30.11.2014 um 23:29 schrieb John Hardin:

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

if($a > 1 && undefined_function())
{

}

in PHP will also not throw an error when $a <= 1

exactly that's what i menat with "from my knowledge the 'if perl_version' should not get touched in that case at all"

signature.asc
Description: OpenPGP digital signature