ports/177097: [PATCH] mail/milter-greylist loses dumpfile following upgrade

[John Marshall]

>Number:         177097
>Category:       ports
>Synopsis:       [PATCH] mail/milter-greylist loses dumpfile following upgrade
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Tue Mar 19 01:40:00 UTC 2013
>Closed-Date:
>Last-Modified:
>Originator:     John Marshall
>Release:        FreeBSD 9.1-RELEASE-p1 amd64
>Organization:
Riverwillow Pty Ltd
>Environment:
System: FreeBSD rwsrv08.mby.riverwillow.net.au 9.1-RELEASE-p1 FreeBSD 9.1-RELEASE-p1 #0 r248445: Mon Mar 18 12:35:45 AEDT 2013 root at rwsrv08.mby.riverwillow.net.au:/usr/obj/usr/src/sys/RWSRV08 amd64


	
>Description:

Following upgrade to 4.4.1, milter-greylist looks for its dumpfile in a
different (non-existent) location...

  milter-greylist: cannot read dumpfile "/usr/local/var/milter-greylist/greylist.db"
  milter-greylist: starting with an empty greylist

The problem arises because the configure.ac defines the default dumpfile
location as ${prefix}/var...  We could patch config files but since the
port already hard-codes the expected dumpfile location in the Makefile,
I suggest we just use the configure argument to pass that location to
the build.  The attached patch addes that configure argument.

I notice that ports/176950 (4.4.3 update) is in progress.  Perhaps this
patch could augment the commit for that PR?

Thank you for maintaining this port.

>How-To-Repeat:
	
>Fix:

	

--- milter-greylist.diff begins here ---
Index: mail/milter-greylist/Makefile
===================================================================
--- mail/milter-greylist/Makefile	(revision 314609)
+++ mail/milter-greylist/Makefile	(working copy)
@@ -74,8 +74,9 @@
 LIBS+=		${PTHREAD_LIBS}
 
 GNU_CONFIGURE=	yes
-CONFIGURE_ARGS+=	--with-user=mailnull --with-libmilter=${MILTERBASE} \
-			--enable-dnsrbl --with-thread-safe-resolver
+CONFIGURE_ARGS+=	--with-user=mailnull --with-libmilter=${MILTERBASE}	\
+			--enable-dnsrbl --with-thread-safe-resolver		\
+			--with-dumpfile=/var/milter-greylist/greylist.db
 CONFIGURE_ENV+=	LIBS="${LIBS}"
 
 ALL_TARGET=	milter-greylist
--- milter-greylist.diff ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: