On 11/26/2014 11:56 AM, John Hardin wrote:
On Wed, 26 Nov 2014, Burnie wrote:
Seems one of the new rules fails to lint.
Perhaps there's a mismatch with the regexp and perl 5.8?
spamassassin 3.3.1
perl 5.8.8
# /usr/bin/sa-update
config: invalid regexp for rule __PDS_FROM_2_EMAILS:
/^\W+([\w+.-]+\@[\w.-]+\.\w\w++)(?:[^\n\w<]{0,80})?<(?!\1)[^\n\s]*\@/i:
Nested quantifiers in regex; marked by <-- HERE in
m/(?i)^\W+([\w+.-]+\@[\w.-]+\.\w\w++ <-- HERE
)(?:[^\n\w<]{0,80})?<(?!\1)[^\n\s]*\@/
channel: lint check of update failed, channel failed
Whoops. The ++ syntax was introduced in perl 5.10
Since there's no way to add a perl version exclusion to that rule I've
commented it out, but I don't know how quickly it will go out. Any
possibility of upgrading to at least perl 5.10.0?
You could try editing your local 72_active.cf to remove the two
PDS_FROM_2_EMAILS rules (subrule and meta):
# header __PDS_FROM_2_EMAILS From =~
/^\W+([\w+.-]+\@[\w.-]+\.\w\w++)(?:[^\n\w<]{0,80})?<(?!\1)[^\n\s]*\@/i
# meta PDS_FROM_2_EMAILS __PDS_FROM_2_EMAILS && !__VIA_ML &&
!__VIA_RESIGNER
I don't think the 72_active is installed because the sa-update fails and
I don't think there is a force command for sa-update without a lint.
Nothing should be broken because it should have aborted the sa-update
run without touching the existing rules and in a day or three he'll get
the rules without that rule (or hopefully with a revised version), etc.
Regards,
KAM
