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


--
 John Hardin KA7OHZ                    http://www.impsec.org/~jhardin/
 jhar...@impsec.org    FALaholic #11174     pgpk -a jhar...@impsec.org
 key: 0xB8732E79 -- 2D8C 34F4 6411 F507 136C  AF76 D822 E6E6 B873 2E79
-----------------------------------------------------------------------
  "Bother," said Pooh as he struggled with /etc/sendmail.cf, "it never
  does quite what I want. I wish Christopher Robin was here."
                                           -- Peter da Silva in a.s.r
-----------------------------------------------------------------------
 29 days until Christmas

Reply via email to