- Fix a typo in PCRE_CONFIGURE_ENABLE knob

- Remove MSDAV option: this is no longer valid compiler flag in pound,
  for quite some time
- Do not pass -I to CFLAGS, it is already added (correctly) to CPPFLAGS
- Move initial CONFIGURE_ARGS after GNU_CONFIGURE where it logically belongs
- Move OPTIONS and helper knobs right above .include <bsd.port.options.mk>
- Remove ".if ${PORT_OPTIONS:MDOCS}" guard, not needed in post-staging
- Slightly beautify files/pound.in and pkg-descr while I'm here

PR:		199339
Submitted by:	maintainer

www/pound/Makefile

@@ -3,7 +3,7 @@
 
 PORTNAME=	pound
 PORTVERSION=	2.7
-PORTREVISION=
+PORTREVISION=	1
 CATEGORIES=	www net
 MASTER_SITES=	http://www.apsis.ch/pound/ \
 		ftp://ftp.nuug.no/pub/anders/distfiles/
@@ -17,32 +17,29 @@ LICENSE_FILE=	${WRKSRC}/GPL.txt
 
 USES=		tar:tgz
 GNU_CONFIGURE=	yes
+CONFIGURE_ARGS=	--disable-tcmalloc --disable-hoard
 USE_OPENSSL=	yes
 USE_RC_SUBR=	${PORTNAME}
 
 SUB_LIST=	NAME=${PORTNAME}
 
-OPTIONS_DEFINE=	DOCS MSDAV PCRE
-OPTIONS_DEFAULT=PCRE
-MSDAV_DESC=	Microsoft WebDAV extension support
-
-PCRE_CONFIGUE_ENABLE=	pcreposix
-MSDAV_CONFIGURE_ENABLE=	msdav
-CONFIGURE_ARGS=		--disable-tcmalloc --disable-hoard
-
 PLIST_FILES=	sbin/pound \
 		sbin/poundctl \
 		man/man8/pound.8.gz \
 		man/man8/poundctl.8.gz
 PORTDOCS=	README FAQ CHANGELOG
 
+OPTIONS_DEFINE=	DOCS PCRE
+OPTIONS_DEFAULT=PCRE
+
+PCRE_CONFIGURE_ENABLE=	pcreposix
+
 .include <bsd.port.options.mk>
 
 .if ${PORT_OPTIONS:MPCRE}
-LIB_DEPENDS+=		libpcre.so:${PORTSDIR}/devel/pcre
-CFLAGS+=		-I${LOCALBASE}/include
-CPPFLAGS+=		-I${LOCALBASE}/include
-LDFLAGS+=		-L${LOCALBASE}/lib
+LIB_DEPENDS+=	libpcre.so:${PORTSDIR}/devel/pcre
+CPPFLAGS+=	-I${LOCALBASE}/include
+LDFLAGS+=	-L${LOCALBASE}/lib
 .endif
 
 do-install:
@@ -50,9 +47,7 @@ do-install:
 		${STAGEDIR}${PREFIX}/sbin
 	${INSTALL_MAN} ${WRKSRC}/pound.8 ${WRKSRC}/poundctl.8 \
 		${STAGEDIR}${MANPREFIX}/man/man8
-.if ${PORT_OPTIONS:MDOCS}
 	${MKDIR} ${STAGEDIR}${DOCSDIR}
 	cd ${WRKSRC} && ${INSTALL_DATA} ${PORTDOCS} ${STAGEDIR}${DOCSDIR}
-.endif
 
 .include <bsd.port.mk>

www/pound/files/pound.in

@@ -10,7 +10,7 @@
 
 # Add the following line to /etc/rc.conf to enable `%%NAME%%':
 #
-#%%NAME%%_enable="YES"
+# %%NAME%%_enable="YES"
 #
 # and be sure to configure %%PREFIX%%/etc/%%NAME%%.conf
 

www/pound/pkg-descr

@@ -1,7 +1,7 @@
 The Pound program is a reverse proxy, load balancer and HTTPS front-end for
 Web server(s). Pound was developed to enable distributing load among several
 Web-servers, and to allow for a convenient SSL wrapper for those Web servers
-that do not offer it natively. Pound is distributed under the GPL - no
+that do not offer it natively. Pound is distributed under the GPL -- no
 warranty, it's free to use, copy and give away.
 
 WWW: http://www.apsis.ch/pound/