svn commit: r442588 - in head/www: nginx nginx-full

[Adam Weinberger]

> On 4 Jun, 2017, at 18:18, Sergey A. Osokin <osa at FreeBSD.org> wrote:
> 
> Hi Bartek and Adam,
> 
> I don't think I can get this, so two questions for you guys:
> o) what was the reason to bump PORTREVISION in www/nginx?
> o) wouldn't it btter to just bump PORTREVISION in www/nginx-full?

Hi Sergey,

I'll give Bartek a chance to explain in more detail, but I supported an nginx bump because it was less complex for the future.

If nginx-full got a bump, then it would need to be bumped every time nginx got bumped, or nginx would have to be bumped by two and nginx-full's PORTREVISION line gets removed, and then the line has to be removed at the next nginx update or reset. At the end of the day, bumping nginx was more straightforward. It triggers an update for everyone else, but becomes less invasive over the long haul.

# Adam


-- 
Adam Weinberger
adamw at adamw.org
https://www.adamw.org


> 
> --
> Sergey Osokin
> 
> On Sun, Jun 04, 2017 at 08:38:26PM +0000, Bartek Rutkowski wrote:
>> Author: robak
>> Date: Sun Jun  4 20:38:26 2017
>> New Revision: 442588
>> URL: https://svnweb.freebsd.org/changeset/ports/442588
>> 
>> Log:
>>  www/nginx and www/nginx-full: remove obsolete modules and bump PORTREVISION
>> 
>>  Some time ago www/nginx master port removed two external modules that
>>  stayed in www/nginx-full slave port options causing pkg build failures.
>>  Thich change removes the modules from www/nginx-full and bumps www/nginx
>>  PORTREVISION to force revision bump for slave port.
>> 
>>  Approved by:	portmgr (adamw)
>> 
>> Modified:
>>  head/www/nginx-full/Makefile
>>  head/www/nginx/Makefile
>> 
>> Modified: head/www/nginx-full/Makefile
>> ==============================================================================
>> --- head/www/nginx-full/Makefile	Sun Jun  4 20:35:48 2017	(r442587)
>> +++ head/www/nginx-full/Makefile	Sun Jun  4 20:38:26 2017	(r442588)
>> @@ -22,7 +22,7 @@ OPTIONS_DEFAULT=DSO DEBUG DEBUGLOG FILE_AIO GOOGLE_PER
>> 		HTTP_SSL HTTP_STATUS HTTP_SUB HTTP_XSLT MAIL MAIL_IMAP \
>> 		MAIL_POP3 MAIL_SMTP MAIL_SSL HTTPV2 NJS STREAM STREAM_SSL \
>> 		THREADS WWW AJP AWS_AUTH CACHE_PURGE CLOJURE CT ECHO \
>> -		FASTDFS HEADERS_MORE HTTP_ACCEPT_LANGUAGE HTTP_ACCESSKEY \
>> +		FASTDFS HEADERS_MORE HTTP_ACCEPT_LANGUAGE \
>> 		HTTP_AUTH_DIGEST HTTP_AUTH_KRB5 HTTP_AUTH_LDAP HTTP_AUTH_PAM \
>> 		HTTP_DAV_EXT HTTP_EVAL HTTP_FANCYINDEX HTTP_FOOTER \
>> 		HTTP_GEOIP2 HTTP_JSON_STATUS HTTP_MOGILEFS HTTP_MP4_H264 \
>> @@ -34,6 +34,6 @@ OPTIONS_DEFAULT=DSO DEBUG DEBUGLOG FILE_AIO GOOGLE_PER
>> 		FORMINPUT GRIDFS ICONV LET LUA MEMC MODSECURITY \
>> 		NAXSI PASSENGER POSTGRES RDS_CSV RDS_JSON REDIS2 \
>> 		RTMP SET_MISC SFLOW SHIBBOLETH SLOWFS_CACHE SMALL_LIGHT \
>> -		SRCACHE STATSD UDPLOG X11 XRID_HEADER XSS
>> +		SRCACHE STATSD UDPLOG X11 XSS
>> 
>> .include "${MASTERDIR}/Makefile"
>> 
>> Modified: head/www/nginx/Makefile
>> ==============================================================================
>> --- head/www/nginx/Makefile	Sun Jun  4 20:35:48 2017	(r442587)
>> +++ head/www/nginx/Makefile	Sun Jun  4 20:38:26 2017	(r442588)
>> @@ -3,7 +3,7 @@
>> 
>> PORTNAME=	nginx
>> PORTVERSION=	1.12.0
>> -PORTREVISION?=	3
>> +PORTREVISION?=	4
>> PORTEPOCH=	2
>> CATEGORIES=	www
>> MASTER_SITES=	http://nginx.org/download/
>>