svn commit: r335523 - in head/www/nginx: . files

[Mathieu Arnold]

Hi,

+--On 2 décembre 2013 19:10:57 +0000 "Sergey A. Osokin" <osa at FreeBSD.org>
wrote:
| Please revert back these wrong changes ASAP.

The PR has been sitting for four and a half months. If it is wrong, you
should have said something there, so, what exactly is wrong with them ?

Regards,

| On Mon, Dec 02, 2013 at 06:14:20PM +0000, Mathieu Arnold wrote:
|> Author: mat
|> Date: Mon Dec  2 18:14:19 2013
|> New Revision: 335523
|> URL: http://svnweb.freebsd.org/changeset/ports/335523
|> 
|> Log:
|>   The patch for adding syslog support sets the wrong define variable
|>   causing syslog support to not be built.
|>   
|>   maintainer timeout.
|>   
|>   PR:		ports/180639
|>   Submitted by:	Dave Duchscher
|> 
|> Modified:
|>   head/www/nginx/Makefile
|>   head/www/nginx/files/extra-patch-syslog_support
|> 
|> Modified: head/www/nginx/Makefile
|> ========================================================================
|> ====== --- head/www/nginx/Makefile	Mon Dec  2 18:08:41 2013	(r335522)
|> +++ head/www/nginx/Makefile	Mon Dec  2 18:14:19 2013	(r335523)
|> @@ -3,6 +3,7 @@
|>  
|>  PORTNAME=	nginx
|>  PORTVERSION=	1.4.4
|> +PORTREVISION=	1
|>  PORTEPOCH=	1
|>  CATEGORIES=	www
|>  MASTER_SITES=	http://nginx.org/download/
|> 
|> Modified: head/www/nginx/files/extra-patch-syslog_support
|> ========================================================================
|> ====== --- head/www/nginx/files/extra-patch-syslog_support	Mon Dec  2
|> 18:08:41 2013	(r335522) +++
|> head/www/nginx/files/extra-patch-syslog_support	Mon Dec  2 18:14:19 2013
|> 	(r335523) @@ -721,7 +721,7 @@ index 54e1c26..2b05157 100644
|>  +    SYSLOG_FACILITY="LOG_DAEMON"
|>  +fi
|>  +if test "${USE_SYSLOG}" = "YES"; then
|> -+    CFLAGS="$CFLAGS -DUSE_SYSLOG -DSYSLOG_FACILITY=${SYSLOG_FACILITY}"
|> ++    CFLAGS="$CFLAGS -DNGX_ENABLE_SYSLOG
|> -DSYSLOG_FACILITY=${SYSLOG_FACILITY}" +fi
|>   
|>   cat << END                                                     >
|>   $NGX_MAKEFILE
| 



-- 
Mathieu Arnold