Date: Mon, 15 Nov 2004 11:45:48 +0100 From: Maxime Ritter <airmax@netlibre.info> To: David du SERRE-TELMON <david@xinus.net> Cc: cmt-spamassassin@someone.dhs.org Subject: Re: Rules du jour Message-ID: <20041115104548.GA29422@sous-bock.netlibre.info> In-Reply-To: <4197B479.7000507@xinus.net> References: <4197B479.7000507@xinus.net>
next in thread | previous in thread | raw e-mail | index | archive | help
--M9NhX3UHpAaciwkO Content-Type: text/plain; charset=iso-8859-15 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Nov 14, 2004 at 07:39:37PM +0000, David du SERRE-TELMON wrote: > Rules du jour doesn't work with SpamAssassin 3.x because of different > sanitity checks add with SA3. > After adding the new rules with Rules du jour, SA complains about errors > in the syntax of the rules : > - description is over 50 chars That's because of rulesets which weren't updated for SA 3.0. Like mine. (but I started shortening the description). > - description exists for non-existent rule This might hide a real bug.=20 > - score set for non-existent rule Yeah, must be with my airmax.cf. I change the score of some rulesets I really don't like, such as : score SARE_FREE_WEBM_FrYahoo 0 (can't understand the guy who wrote this kind of rule, because these free webmails aren't evil). Consequently, if you use airmax.cf (or my french ruleset), but you don't us= e=20 the ruleset which contains the rules I desactive (such as SARE header), the= =20 linting process shows many of this warning. So, in the case you use airmax.cf, it might be safely ignored.=20 IMHO, if the new given score is zero (ie desactivating the rule), the non-existent rule warning should not be shown. Or maybe adding a way to change the score only if it exist to SA ? > I think these warnings can be safely ignored. This can be done by > updating the SA Perl parser and disable the call of the function lint_che= ck. No, don't do this ! The linting process is very usefull for some real bugs (like an invalid regex). If you want to do something, you might just disable the "descriptio= n=20 is over 50 chars" and the "score set for non-existent rule" checks. > I think, this tip should be add in the README file of Rule du Jour and > in the port of Rule du jour. This change the code of SA, I don't think > we could IMHO, the proper way to do it, would be to add an option to SA code to disable/enable these checks. > Also, this message is for Chris, it could be a good idea to add this > rule in the default script : > http://maxime.ritter.eu.org/Spam/french_rules.cf Of course, I agree, it would be a good idea. But please, don't do automatic= =20 downloading on this host. Use the mirror instead : -> http://airmex.nerim.net/rule-get/french_rules.cf The mirror is upgraded in real time. > This one is also very good for all spams : > http://maxime.ritter.eu.org/Spam/airmax.cf -> http://airmex.nerim.net/rule-get/airmax.cf --=20 Maxime Ritter - http://maxime.ritter.eu.org - maxime@ritter.eu.org --M9NhX3UHpAaciwkO Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.4 (GNU/Linux) iD8DBQFBmIjbGiOLU3oQf54RAo8PAJ0eaOYJTWptrCL7mZb17x/qYTJnXwCfZagh dHwup6W9r8lXOndpCcwx3Qk= =+pnM -----END PGP SIGNATURE----- --M9NhX3UHpAaciwkO--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20041115104548.GA29422>