From owner-cvs-ports@FreeBSD.ORG Tue Nov 22 16:19:26 2011 Return-Path: Delivered-To: cvs-ports@FreeBSD.org Received: by hub.freebsd.org (Postfix, from userid 1033) id 9E5F2106566C; Tue, 22 Nov 2011 16:19:26 +0000 (UTC) Date: Tue, 22 Nov 2011 16:19:26 +0000 From: Alexey Dokuchaev To: Pietro Cerutti Message-ID: <20111122161926.GA88372@FreeBSD.org> References: <201111221258.pAMCw8fZ040772@repoman.freebsd.org> <20111122142709.GA55372@FreeBSD.org> <20111122151553.GJ22065@gahrfit.gahr.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <20111122151553.GJ22065@gahrfit.gahr.ch> User-Agent: Mutt/1.4.2.1i Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/graphics/gegl Makefile X-BeenThere: cvs-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Nov 2011 16:19:26 -0000 On Tue, Nov 22, 2011 at 04:15:53PM +0100, Pietro Cerutti wrote: > On 2011-Nov-22, 14:27, Alexey Dokuchaev wrote: > > On Tue, Nov 22, 2011 at 12:58:08PM +0000, Pietro Cerutti wrote: > > > gahr 2011-11-22 12:58:08 UTC > > > > > > Modified files: > > > graphics/gegl Makefile > > > Log: > > > - Disable VALA support > > > > Perhaps better message would be "Disable VALA support as it appears broken". > > Yes, thanks a lot. Perhaps a bit of explanation is in order. Generally speaking, commit logs should contain about 1/3 of description of what had been done and remaining 2/3 should be devoted to the reasoning (why was it done). In this particular case, it's completely unclear what's exactly wrong with VALA support that you'd had to disable it (although I definitely support having generally useless stuff disabled by default); I'd had to look through the PR to find mentions of build breakage. ./danfe