From owner-svn-ports-all@freebsd.org Tue Jun 5 15:03:14 2018 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 20BA8FE946D; Tue, 5 Jun 2018 15:03:14 +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 CBBA87D952; Tue, 5 Jun 2018 15:03:13 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id C407DA384; Tue, 5 Jun 2018 15:03:13 +0000 (UTC) Date: Tue, 5 Jun 2018 15:03:13 +0000 From: Alexey Dokuchaev To: Mathieu Arnold Cc: Sunpoet Po-Chuan Hsieh , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r471577 - head/databases/p5-DMOZ-ParseRDF Message-ID: <20180605150313.GA17615@FreeBSD.org> References: <201806041801.w54I1fqw059369@repo.freebsd.org> <20180605134236.GA3309@FreeBSD.org> <20180605145320.b7fdhbfadroamo6d@ogg.in.absolight.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180605145320.b7fdhbfadroamo6d@ogg.in.absolight.net> User-Agent: Mutt/1.9.5 (2018-04-13) X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.26 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: Tue, 05 Jun 2018 15:03:14 -0000 On Tue, Jun 05, 2018 at 04:53:20PM +0200, Mathieu Arnold wrote: > On Tue, Jun 05, 2018 at 01:42:36PM +0000, Alexey Dokuchaev wrote: > > On Mon, Jun 04, 2018 at 06:01:41PM +0000, Sunpoet Po-Chuan Hsieh wrote: > > > New Revision: 471577 > > > URL: https://svnweb.freebsd.org/changeset/ports/471577 > > > > > > Log: > > > Add LICENSE > > > > > > - Add NO_ARCH > > > - Update WWW > > > > Is there any reason you format commit logs like that? It's kind of hard > > to read. Why not simply: > > > > - Add LICENSE > > - Add NO_ARCH > > - Update WWW > > Or, even better, something like this: > > ``` > This port does not install any architecture dependant files. > > While there, add LICENSE and update WWW, search.cpan.org is going away. > ``` > > Or something else that actually describe WHY, and not an english > translation of the diff. Yes, of course, that would be even better, but given how sunpoet@ is usually un^Wresponsive to the feedback, I've started with smaller step. ./danfe