From owner-svn-ports-head@freebsd.org Sun May 6 10:50:13 2018 Return-Path: Delivered-To: svn-ports-head@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 E7BBFFB5D27; Sun, 6 May 2018 10:50:12 +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 4CD9A75728; Sun, 6 May 2018 10:50:12 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id 443BF6932; Sun, 6 May 2018 10:50:12 +0000 (UTC) Date: Sun, 6 May 2018 10:50:12 +0000 From: Alexey Dokuchaev To: Kurt Jaeger Cc: Koichiro Iwao , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r469188 - head/ports-mgmt/pkg_replace Message-ID: <20180506105012.GA39087@FreeBSD.org> References: <201805061015.w46AF3HQ070595@repo.freebsd.org> <20180506102749.GA58637@FreeBSD.org> <20180506103420.GW25490@fc.opsec.eu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180506103420.GW25490@fc.opsec.eu> User-Agent: Mutt/1.9.2 (2017-12-15) X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.25 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: Sun, 06 May 2018 10:50:13 -0000 On Sun, May 06, 2018 at 12:34:20PM +0200, Kurt Jaeger wrote: > > On Sun, May 06, 2018 at 10:15:03AM +0000, Koichiro Iwao wrote: > > > New Revision: 469188 > > > URL: https://svnweb.freebsd.org/changeset/ports/469188 > > > > > > Log: > > > ports-mgmt/pkg_replace: Update to 20180424 > > > > > > - Cleanup code > > > - Speed up and strictly check package's dependencies > > > - While here, pet portlint by using pkg-plist instead of PLIST_FILES > > > > Could you explain what did you achieve by moving three-line PLIST_FILES > > into pkg-plist? How can "pet portlint" be a reason for a change while > > we generally encourage converting <6-line pkg-plist's into PLIST_FILES? > > There was a problem if the pkg-list stayed in PLIST_FILES > (the build in poudriere failed with a signal 6 in make package). > > Koichiro is working to reproduce/report this, until then... *facepalm*. This poudriere-driven development is more and more disturbing. You didn't attempt to understand the problem, you didn't attempt to fix the PLIST_FILES (remove bogus %%ETCDIR%% since this is Makefile), you didn't ask for a peer review. meta@ pessimized the port, provided wrong commit log (it tells *nothing* about poudriere failing with a signal 6 in make package, it says "pet portlint"), and you've approved it like there's nothing wrong with all of this. :-( ./danfe