From owner-svn-ports-head@freebsd.org  Wed Mar 22 20:18:36 2017
Return-Path: <owner-svn-ports-head@freebsd.org>
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 EECA2D178C0;
 Wed, 22 Mar 2017 20:18:36 +0000 (UTC)
 (envelope-from danfe@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 D0BE01842;
 Wed, 22 Mar 2017 20:18:36 +0000 (UTC)
 (envelope-from danfe@freebsd.org)
Received: by freefall.freebsd.org (Postfix, from userid 1033)
 id 2F30DE3D; Wed, 22 Mar 2017 20:18:36 +0000 (UTC)
Date: Wed, 22 Mar 2017 20:18:36 +0000
From: Alexey Dokuchaev <danfe@FreeBSD.org>
To: Kirill Ponomarev <kp@krion.cc>
Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org,
 svn-ports-head@freebsd.org
Subject: Re: svn commit: r436712 - in head/net/zerotier: . files
Message-ID: <20170322201836.GA21121@FreeBSD.org>
References: <201703221812.v2MICvxf040839@repo.freebsd.org>
 <20170322182712.GA96503@FreeBSD.org>
 <20170322195748.5bltosnrvbzsvdte@vein>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170322195748.5bltosnrvbzsvdte@vein>
User-Agent: Mutt/1.7.1 (2016-10-04)
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
 <svn-ports-head.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-ports-head>, 
 <mailto:svn-ports-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-ports-head/>
List-Post: <mailto:svn-ports-head@freebsd.org>
List-Help: <mailto:svn-ports-head-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-ports-head>,
 <mailto:svn-ports-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 22 Mar 2017 20:18:37 -0000

On Wed, Mar 22, 2017 at 08:57:48PM +0100, Kirill Ponomarev wrote:
> On 03/22, Alexey Dokuchaev wrote:
> > On Wed, Mar 22, 2017 at 06:12:57PM +0000, Kirill Ponomarew wrote:
> > > New Revision: 436712
> > > URL: https://svnweb.freebsd.org/changeset/ports/436712
> > > 
> > > Log:
> > >   Update net/zerotier to 1.2.2
> > >   
> > >   PR:		217838
> > >   Submitted by:	dch@skunkwerks.at
> > >   Approved by:	mat (mentor)
> > > [...]
> > >  
> > > -MAINTAINER=	corsmith@gmail.com
> > > +MAINTAINER=	dch@skunkwerks.at
> > 
> > This part was not reflected in the log.
> 
> This part was reflected in PR.

It should also be reflected in the commit log.

> > > -USES=		gmake
> > > +USES=		gmake compiler:c11
> > 
> > Nor was this one.
>  
> This part is reflected in PR as well.

Ditto.

> > > -OPTIONS_DEFINE=	CONTROLLER
> > > -
> > > -CONTROLLER_DESC=	Network management controller microservice support
> > > -CONTROLLER_USES=	sqlite
> > > -CONTROLLER_MAKE_ARGS=	ZT_ENABLE_NETWORK_CONTROLLER=1
> > 
> > Why this part was removed after I've added it just several days ago in
> > r435612 (see PR 215031)?  If this feature is no longer valid it should
> > have been reflected in the commit log.  Please do not destroy work of
> > others without at least giving a sound reason.
> 
> Feel free to coordinate your work with current (new) maintainer and
> commit needed changes.

Wrong.  Someone (PR 215031) had contributed their work, someone else (me)
reviewed it and committed it.  One just should not in good will step over
and kill it with no explanation.  This kind of attitude and action would
turn away people from making any further contributions and give us pretty
bad looks.

./danfe