I did a couple of times. :(
> -----Original Message-----
> From: Billy Huddleston [mailto:[EMAIL PROTECTED]
> Sent: Wednesday, November 01, 2006 9:20 PM
> To: Dylan Bouterse; users@spamassassin.apache.org
> Subject: Re: Relay Checker Plugin (code review please?)
>
> You may want to download new RelayChecker.pm file... you may have
messed
> it
> up previously..
>
> If you still have problems let me know..
>
> ----- Original Message -----
> From: "Dylan Bouterse" <[EMAIL PROTECTED]>
> To: <users@spamassassin.apache.org>
> Sent: Wednesday, November 01, 2006 6:39 PM
> Subject: RE: Relay Checker Plugin (code review please?)
>
>
> > -----Original Message-----
> > From: John D. Hardin [mailto:[EMAIL PROTECTED]
> > Sent: Wednesday, November 01, 2006 5:05 PM
> > To: Dylan Bouterse
> > Cc: users@spamassassin.apache.org
> > Subject: RE: Relay Checker Plugin (code review please?)
> >
> > On Wed, 1 Nov 2006, Dylan Bouterse wrote:
> >
> > > # header RELAY_CHECKER eval:relay_checker()
> > > # describe RELAY_CHECKER Check relay for DNS/Hostname
issues.
> > > to:
> > > if ($nordns) {
> > >
> > > and when I run --lint I get the following errors:
> > >
> > > /etc/mail/spamassassin/RelayChecker.pm line 44, near "27 @@
> >
> > ...how exactly did you apply the patch? From the contents of that
> > error message it looks like you just inserted the patch text into
the
> > source file...
> >
> > Take a look at "man patch".
> >
> > (Sorry if you did do that, but that error message is really
suggestive
> > of improper procedure.)
> >
>
> I have never used the patch command and was not aware of it. Thank you
> for pointing me in the right direction. I was able to patch my
> RelayChecker.cf file using the patch command and the provided patch
for
> that file but I am getting errors when trying to patch the
> RelayChecker.pm file.
>
> [EMAIL PROTECTED] spamassassin]# patch -i RelayChecker.pm.patch
> RelayChecker.pm
> missing header for unified diff at line 3 of patch
> patching file RelayChecker.pm
> Hunk #3 succeeded at 102 with fuzz 1.
> missing header for unified diff at line 77 of patch
> can't find file to patch at input line 77
> Perhaps you should have used the -p or --strip option?
> The text leading up to this was:
> --------------------------
> | if (! defined($name)) {
> | # the PTR record leads to a host that doesn't resolve in
DNS
> | Mail::SpamAssassin::Plugin::dbg("RelayChecker: badrdns");
> |- $badrdns = 1;
> |+ $badrdns = $badrdns_score;
> | }
> | else {
> | Mail::SpamAssassin::Plugin::dbg("RelayChecker: name is
> $name"); @@ -96,7 +123,7 @@
> | # the hostname in the PTR record does resolve, but that
> hostname
> | # doesn't have $ip as one of its IP addresses
> | Mail::SpamAssassin::Plugin::dbg("RelayChecker: baddns");
> |- $baddns = 1;
> |+ $baddns = $baddns_score;
> | }
> | else {
> | ($a, $b, $c, $d) = split(/\./, $ip); # decimal octets @@
> -124,7 +151,7 @@
> | # in hex or decimal form ... or the entire thing in
> decimal
> | # probably a spambot since this is an untrusted relay
> | Mail::SpamAssassin::Plugin::dbg("RelayChecker:
> ipinhostname");
> |- $ipinhostname = 1;
> |+ $ipinhostname = $ipinhostname_score;
> | }
> | if ($hostname =~
> |
>
/(cable|catv|client|ddns|dhcp|dial-?up|dip|dsl|dynamic|ppp)\S*\.\S+\.\S+
> $/
> --------------------------
