From owner-svn-ports-all@freebsd.org Fri Dec 15 19:40:55 2017 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 6E1D7E8AAC1; Fri, 15 Dec 2017 19:40:55 +0000 (UTC) (envelope-from osa@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2610:1c1:1:6074::16:84]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 456CE375A; Fri, 15 Dec 2017 19:40:55 +0000 (UTC) (envelope-from osa@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 975) id 5F55F18A24; Fri, 15 Dec 2017 19:40:54 +0000 (UTC) Date: Fri, 15 Dec 2017 19:40:54 +0000 From: "Sergey A. Osokin" To: "Tobias C. Berner" Cc: Jochen Neumeister , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r456286 - head/www/nginx Message-ID: <20171215194054.GG81079@FreeBSD.org> References: <201712141007.vBEA74si099181@repo.freebsd.org> <20171215004922.GD81079@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Dec 2017 19:40:55 -0000 Hi Tobias, objection because third-party fancyindex module update doesn't affect the final package anyhow. -- Sergey A. Osokin On Fri, Dec 15, 2017 at 08:24:51PM +0100, Tobias C. Berner wrote: > Hi Sergey > > Jochen did it correctly -- the porters handbook [1] clearly states: > ""PORTREVISION must be increased each time a change is made to the port > that changes the generated package in any way. > *That includes changes that only affect a package built with non-default > options > ."* > > > mfg Tobias > > [1] https://www.freebsd.org/doc/en/books/porters-handbook/ > book.html#makefile-portrevision > > On 15 December 2017 at 01:49, Sergey A. Osokin wrote: > > > Hi Jochen, > > > > fancyindex is a third-party module, it isn't enabled by default, > > so no reason to bump PORTREVISION in this case. > > > > -- > > Sergey A. Osokin > > > > On Thu, Dec 14, 2017 at 10:07:04AM +0000, Jochen Neumeister wrote: > > > Author: joneum > > > Date: Thu Dec 14 10:07:04 2017 > > > New Revision: 456286 > > > URL: https://svnweb.freebsd.org/changeset/ports/456286 > > > > > > Log: > > > www/nginx: Update fancyindex to 0.4.2 > > > > > > This release contains an important fix which can cause Nginx > > > to crash when a directory contains zero-sized (empty) files. > > > This bug has been present in all previous releases, > > > and all users are strongly encouraged to update to version 0.4.2. > > > > > > PR: 224237 > > > Submitted by: wosch > > > Approved by: tz (mentor) > > > MFH: 2017Q4 > > > Differential Revision: https://reviews.freebsd.org/D13486 > > > > > > Modified: > > > head/www/nginx/Makefile > > > head/www/nginx/distinfo > > > > > > Modified: head/www/nginx/Makefile > > > ============================================================ > > ================== > > > --- head/www/nginx/Makefile Thu Dec 14 09:31:09 2017 (r456285) > > > +++ head/www/nginx/Makefile Thu Dec 14 10:07:04 2017 (r456286) > > > @@ -3,7 +3,7 @@ > > > > > > PORTNAME= nginx > > > PORTVERSION= 1.12.2 > > > -PORTREVISION?= 2 > > > +PORTREVISION?= 3 > > > PORTEPOCH= 2 > > > CATEGORIES= www > > > MASTER_SITES= http://nginx.org/download/ > > > @@ -529,7 +529,7 @@ CONFIGURE_ARGS+=--add-module=${WRKSRC_eval} > > > .endif > > > > > > .if ${PORT_OPTIONS:MHTTP_FANCYINDEX} > > > -NGINX_FANCYINDEX_VERSION= 0.4.1 > > > +NGINX_FANCYINDEX_VERSION= 0.4.2 > > > GH_ACCOUNT+= aperezdc:fancyindex > > > GH_PROJECT+= ngx-fancyindex:fancyindex > > > GH_TAGNAME+= v${NGINX_FANCYINDEX_VERSION}:fancyindex > > > > > > Modified: head/www/nginx/distinfo > > > ============================================================ > > ================== > > > --- head/www/nginx/distinfo Thu Dec 14 09:31:09 2017 (r456285) > > > +++ head/www/nginx/distinfo Thu Dec 14 10:07:04 2017 (r456286) > > > @@ -1,4 +1,4 @@ > > > -TIMESTAMP = 1512144757 > > > +TIMESTAMP = 1513235728 > > > SHA256 (nginx-1.12.2.tar.gz) = 305f379da1d5fb5aefa79e45c82985 > > 2ca6983c7cd2a79328f8e084a324cf0416 > > > SIZE (nginx-1.12.2.tar.gz) = 981687 > > > SHA256 (ngx_cache_purge-2.3.tar.gz) = 279e0d8a46d3b1521fd43b3f78bc1c > > 08b263899142a7cc5058c1c0361a92c89c > > > @@ -47,8 +47,8 @@ SHA256 (arut-nginx-dav-ext-module-v0.0.3_GH0.tar.gz) = > > > SIZE (arut-nginx-dav-ext-module-v0.0.3_GH0.tar.gz) = 6260 > > > SHA256 (openresty-nginx-eval-module-342c812_GH0.tar.gz) = > > c45f6438474593f0cf5229160327d20c2f2d72e9c0b5c8cf9e0ef8d5295de177 > > > SIZE (openresty-nginx-eval-module-342c812_GH0.tar.gz) = 14183 > > > -SHA256 (aperezdc-ngx-fancyindex-v0.4.1_GH0.tar.gz) = > > 2b00d8e0ad2a67152a9cee7b7ee67990c742d501412df912baaf1eee9bb6dc71 > > > -SIZE (aperezdc-ngx-fancyindex-v0.4.1_GH0.tar.gz) = 21130 > > > +SHA256 (aperezdc-ngx-fancyindex-v0.4.2_GH0.tar.gz) = > > 8327150864ca267b735d550d3304030efbbd863fdddfe0a94e970f249a8827ee > > > +SIZE (aperezdc-ngx-fancyindex-v0.4.2_GH0.tar.gz) = 22047 > > > SHA256 (alibaba-nginx-http-footer-filter-1.2.2_GH0.tar.gz) = > > 3493b54460c59370f9f60c6e662862752f1920fc6e684f7a66bb2b3260692813 > > > SIZE (alibaba-nginx-http-footer-filter-1.2.2_GH0.tar.gz) = 3934 > > > SHA256 (leev-ngx_http_geoip2_module-2.0_GH0.tar.gz) = > > ebb4652c4f9a2e1ee31fddefc4c93ff78e651a4b2727d3453d026bccbd708d99 > > > > > > >