svn commit: r456286 - head/www/nginx

[Mathieu Arnold]

Le 15/12/2017 à 22:36, Sergey A. Osokin a écrit :
> On Fri, Dec 15, 2017 at 10:04:58PM +0100, Mathieu Arnold wrote:
>> Le 15/12/2017 ?? 20:40, Sergey A. Osokin a ??crit :
>>> Hi Tobias,
>>>
>>> objection because third-party fancyindex module update doesn't affect
>>> the final package anyhow.
>> You mean that if you enable the fancyindex option, it is not compiled in
>> ? I do not understand.
> I've provided my explanation in previous email, so again - since third-party
> fancyindex module isn't enabled by default, the update of the module
> doesn't affect the final www/nginx package and this is why the PORTREVISION
> bump is needless here.  Hope it's clear.

Let me quote what the porter's handbook say, again, please read it very
carefuly:

   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.

This means that if you change the port so that what gets installed,
whatever the default options are, is changed, you MUST bump PORTREVISION.

I do not understand how you constantly fail to understand this.

-- 
Mathieu Arnold