From owner-svn-ports-head@freebsd.org Sat Jan 6 13:41:32 2018 Return-Path: Delivered-To: svn-ports-head@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 596D8EB9F31; Sat, 6 Jan 2018 13:41:32 +0000 (UTC) (envelope-from osa@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (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 387956E675; Sat, 6 Jan 2018 13:41:32 +0000 (UTC) (envelope-from osa@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 975) id CFD1FFF0A; Sat, 6 Jan 2018 13:41:30 +0000 (UTC) Date: Sat, 6 Jan 2018 13:41:30 +0000 From: "Sergey A. Osokin" To: Mathieu Arnold Cc: v@fatpipi.com, Jochen Neumeister , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r458000 - head/www/nginx Message-ID: <20180106134130.GA39725@FreeBSD.org> References: <201801031933.w03JXLGa033758@repo.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.9.2 (2017-12-15) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 06 Jan 2018 13:41:32 -0000 On Thu, Jan 04, 2018 at 08:30:55AM +0100, Mathieu Arnold wrote: > Le 04/01/2018 ?? 02:56, Vanilla Hsu a ??critš: > > auth-digestš is not default module, so you don't need to bump > > PORTREVISION. > > Yes you do. > To quote > https://www.freebsd.org/doc/en/books/porters-handbook/makefile-naming.html#makefile-portrevision > : > > 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. I've tried to find a commit in doc area to better understand who and why did this change. Another question is why this so important change hasn't been discussed and why committers and community haven't been notified about that in advance. Thanks. -- Sergey A. Osokin