From owner-svn-ports-head@freebsd.org Wed Jun 7 22:06:15 2017 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 0F6E2D7E239; Wed, 7 Jun 2017 22:06:15 +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 DC69E6D6; Wed, 7 Jun 2017 22:06:14 +0000 (UTC) (envelope-from osa@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 975) id 2F822CD2B; Wed, 7 Jun 2017 22:06:14 +0000 (UTC) Date: Wed, 7 Jun 2017 22:06:14 +0000 From: "Sergey A. Osokin" To: Bartek Rutkowski Cc: Adam Weinberger , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r442588 - in head/www: nginx nginx-full Message-ID: <20170607220614.GB55217@FreeBSD.org> References: <201706042038.v54KcQMf001482@repo.freebsd.org> <20170605001807.GA55217@FreeBSD.org> <99D58682-8825-417C-81F8-EDC541D31713@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <99D58682-8825-417C-81F8-EDC541D31713@FreeBSD.org> User-Agent: Mutt/1.8.2 (2017-04-18) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.23 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: Wed, 07 Jun 2017 22:06:15 -0000 On Tue, Jun 06, 2017 at 02:32:17PM +0100, Bartek Rutkowski wrote: > > On 5 Jun 2017, at 01:18, Sergey A. Osokin 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, > > Let me explain it quickly: some time ago you've removed two external modules from www/nginx port, which is a master for www/nginx-full. The www/nginx-full had them in default options, what caused port/pkg build failures and to fix these I needed to remove these two no longer existing modules from default options. After doing so, since it *does* change the contents of the package, I needed to bump the PORTREVISION of www/nginx-full and there were few ways of doing so, but none of them was easy/simple as they were creating even complex scenarios in future bumps/updates, so, after consulting possible solutions with portmgr members, I've chosen one, that while not ideal, have solved the issue for now without creating other issues in future, that is to bump the master www/nginx revision. > > Hope that helps. Hi Bartek, Please don't bump PORTREVISION on www/nginx when you need to do so in www/nginx-full. -- Sergey Osokin