RE: Lint problem with KAM.cf

2021-09-02 Thread Rick Cooper
It lints fine now Rick _ From: Kevin A. McGrail [mailto:kmcgr...@apache.org] Sent: Wednesday, September 01, 2021 5:43 PM To: SA Mailing list Subject: Re: Lint problem with KAM.cf I published a fix for the KAM SHORT A few hours ago. Please let me know how it's working for you

Re: Lint problem with KAM.cf

2021-09-01 Thread Kevin A. McGrail
I published a fix for the KAM SHORT A few hours ago. Please let me know how it's working for you. On Tue, Aug 31, 2021, 23:48 Kevin A. McGrail wrote: > > On 8/31/2021 12:57 PM, Jared Hall wrote: > > 2) OTOH, what's the point of sa-update doing versioning if nobody uses > it? > > It's there to

Re: Lint problem with KAM.cf

2021-08-31 Thread Kevin A. McGrail
On 8/31/2021 12:57 PM, Jared Hall wrote: 2) OTOH, what's the point of sa-update doing versioning if nobody uses it? It's there to support different rulesets but the project as a whole found it better to do more complicated things when rules proved incompatible to encapsulate them in

Re: Lint problem with KAM.cf

2021-08-31 Thread Jared Hall
Rick Cooper wrote: It would appear you are using the SA git trunk version of the plugin (Andrew Colin Kissa informed me of the existence) which changes a LOT of the api. Near as I can tell the old module will work with your rules by adding the following at about line 489 of the

RE: Lint problem with KAM.cf

2021-08-31 Thread Rick Cooper
Cooper Cc: SA Mailing list Subject: Re: Lint problem with KAM.cf We will take a look. We check with lint for every publication but maybe there's a condition we missed or a spelling issue. Thanks for bringing it up. KAM On Mon, Aug 30, 2021, 15:31 Rick Cooper wrote: This have been going

Re: Lint problem with KAM.cf

2021-08-31 Thread Andrew Colin Kissa
Hi There is a new DecodeShortURLs in Spamassassin trunk, the API has changed from the one in the original module on GitHub. The new builtin module has the short_url function but the original module uses short_url_tests, the original module does not have a short_url function thus the error

Re: Lint problem with KAM.cf

2021-08-30 Thread Mike Grau
+1 Same issue here. On 8/30/21 14:31, Rick Cooper wrote: > This have been going on a while but I haven't had time to addresses. > When the KAM rules are updated I see the following lint warning > warn: rules: error: unknown eval 'short_url' for __KAM_SHORT > > Near as I can tell I am running

Re: Lint problem with KAM.cf

2021-08-30 Thread Kevin A. McGrail
We will take a look. We check with lint for every publication but maybe there's a condition we missed or a spelling issue. Thanks for bringing it up. KAM On Mon, Aug 30, 2021, 15:31 Rick Cooper wrote: > This have been going on a while but I haven't had time to addresses. > When the KAM rules

Lint problem with KAM.cf

2021-08-30 Thread Rick Cooper
This have been going on a while but I haven't had time to addresses. When the KAM rules are updated I see the following lint warning warn: rules: error: unknown eval 'short_url' for __KAM_SHORT Near as I can tell I am running the latest DecodeShortURLs.pm but the site says it's being merged