From owner-svn-ports-head@freebsd.org Thu Aug 27 11:33:16 2015 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 1EBC19C3944; Thu, 27 Aug 2015 11:33:16 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from vps.amdmi3.ru (vps.amdmi3.ru [109.234.38.216]) by mx1.freebsd.org (Postfix) with ESMTP id DB3AF2E9; Thu, 27 Aug 2015 11:33:15 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from hive.panopticon (unknown [78.153.152.119]) by vps.amdmi3.ru (Postfix) with ESMTPS id 0821EB059C; Thu, 27 Aug 2015 14:33:15 +0300 (MSK) Received: from hades.panopticon (hades.panopticon [192.168.0.32]) by hive.panopticon (Postfix) with ESMTP id CD62CBFF; Thu, 27 Aug 2015 14:30:20 +0300 (MSK) Received: by hades.panopticon (Postfix, from userid 1000) id C176C1238B0; Thu, 27 Aug 2015 14:33:01 +0300 (MSK) Date: Thu, 27 Aug 2015 14:33:01 +0300 From: Dmitry Marakasov To: marino@freebsd.org Cc: Alexey Dokuchaev , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r395409 - head/astro/geographiclib Message-ID: <20150827113301.GH41937@hades.panopticon> References: <201508270944.t7R9ia0R067164@repo.freebsd.org> <20150827100953.GA78107@FreeBSD.org> <20150827105912.GF41937@hades.panopticon> <55DEEFC2.40309@marino.st> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <55DEEFC2.40309@marino.st> User-Agent: Mutt/1.5.23 (2014-03-12) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.20 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: Thu, 27 Aug 2015 11:33:16 -0000 * John Marino (freebsd.contact@marino.st) wrote: > >>> New Revision: 395409 > >>> URL: https://svnweb.freebsd.org/changeset/ports/395409 > >>> > >>> [...] > >>> GRAVITY_NAMES:= egm84 egm96 egm2008 wgs84 grs80 > >>> -MAGNETIC_NAMES:= wmm2010 wmm2015 igrf11 igrf12 emm2010 > >>> +MAGNETIC_NAMES:= wmm2010 wmm2015 igrf11 igrf12 emm2010 emm2015 > >> > >> Why := (assignment with immediate expansion) is used here, esp. for constant > >> values? > > > > Ask maintainer. > > I thought it had been clearly established (by mat@ IIRC) that this kind > of response is not acceptable as a reason for ports mistakes. The > committer is supposed to review the port he or she is committing and > catch errors. The maintainers simply don't have the same level of > experience as committers do. I'll repeat the same thing I've said back then: I don't see it as a mistake, that is exactly why I've committed it without modification. "There's no difference whether variable is assigned with or without expansion as long as the assigned value does not contain other variable references" - that's what I'd answer if you asked _why_ I committed it as-is, but you've asked another question, and that question may only be directed to maintainer. -- Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D amdmi3@amdmi3.ru ..: jabber: amdmi3@jabber.ru http://amdmi3.ru