Milter-greylist

First of all, great program!

I ran into some difficulty when first installing milter-greylisting 
because of my lack of knowledge.  Had I been more knowledgeable, I would 
have realized that the suggestion to append the lines

INPUT_MAIL_FILTER(`greylist',`S=local:/var/milter-greylist/milter-greylist.sock')
define(`confMILTER_MACROS_CONNECT', `j, {if_addr}')
define(`confMILTER_MACROS_HELO', `{verify}, {cert_subject}')
define(`confMILTER_MACROS_ENVFROM', `i, {auth_authen}')
define(`confMILTER_MACROS_ENVRCPT', `{greylist}')

to the end of the sendmail.mc file would not work, not because they 
might conflict with other milter macros (I had no other milters, so I 
thought it was safe), but because the lines don't end with 'dnl'.

As I know next-to-nothing about M4, the omission was far from obvious.

In any case, assuming milter-greylist.m4 is kept up-to-date, I see no 
reason to reference those lines.  Instead, it ought to just say to append

include('/path/to/milter-greylist/milter-greylist.m4')dnl

where '/path/to/milter-greylist' should be changed to match the 
appropriate directory.

-- 
_______________________________________________________________
Bob Smith - bsmith@... - http://www.sudleyplace.com

Bob Smith <bsmith@...> wrote:

> I ran into some difficulty when first installing milter-greylisting 
> because of my lack of knowledge.  Had I been more knowledgeable, I would
> have realized that the suggestion to append the lines
> 
>
INPUT_MAIL_FILTER(`greylist',`S=local:/var/milter-greylist/milter-greyli
st.sock')
> define(`confMILTER_MACROS_CONNECT', `j, {if_addr}')
> define(`confMILTER_MACROS_HELO', `{verify}, {cert_subject}')
> define(`confMILTER_MACROS_ENVFROM', `i, {auth_authen}')
> define(`confMILTER_MACROS_ENVRCPT', `{greylist}')
> 
> to the end of the sendmail.mc file would not work, not because they 
> might conflict with other milter macros (I had no other milters, so I
> thought it was safe), but because the lines don't end with 'dnl'.

Well, the README says: "Here again, confMILTER_MACROS_* are shared with
other milters, so you need to merge the definitions with what others
milters require. Just copying the lines below is likely to cause other
milters to malfunction."
 
> As I know next-to-nothing about M4, the omission was far from obvious.
> 
> In any case, assuming milter-greylist.m4 is kept up-to-date, I see no
> reason to reference those lines.  Instead, it ought to just say to append
> 
> include('/path/to/milter-greylist/milter-greylist.m4')dnl

You just add this at the bottom of sendmail.cf and everything magically
works?

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu@...