From owner-svn-ports-all@freebsd.org Fri Feb 7 01:05:59 2020 Return-Path: Delivered-To: svn-ports-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 1682323FBC1; Fri, 7 Feb 2020 01:05:59 +0000 (UTC) (envelope-from linimon@lonesome.com) Received: from mail.soaustin.net (mail.soaustin.net [18.222.6.11]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mail.soaustin.net", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 48DHDQ5Dkvz3wdf; Fri, 7 Feb 2020 01:05:58 +0000 (UTC) (envelope-from linimon@lonesome.com) Received: from lonesome.com (unknown [18.188.142.31]) by mail.soaustin.net (Postfix) with ESMTPSA id 969A713E42; Fri, 7 Feb 2020 01:05:57 +0000 (UTC) Date: Fri, 7 Feb 2020 01:05:56 +0000 From: Mark Linimon To: Mathieu Arnold Cc: Emmanuel Vadot , Antoine Brodin , ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r525136 - head/devel Message-ID: <20200207010556.GC15808@lonesome.com> References: <202002040936.0149aI3E088204@repo.freebsd.org> <20200204104020.cba025dc63af413d7688fd8d@bidouilliste.com> <20200204122429.afyvp4w5xx3kb4wk@ogg.in.absolight.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200204122429.afyvp4w5xx3kb4wk@ogg.in.absolight.net> User-Agent: Mutt/1.5.21 (2010-09-15) X-Rspamd-Queue-Id: 48DHDQ5Dkvz3wdf X-Spamd-Bar: / Authentication-Results: mx1.freebsd.org; dkim=none; dmarc=none; spf=none (mx1.freebsd.org: domain of linimon@lonesome.com has no SPF policy when checking 18.222.6.11) smtp.mailfrom=linimon@lonesome.com X-Spamd-Result: default: False [-0.27 / 15.00]; ARC_NA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; NEURAL_HAM_MEDIUM(-0.83)[-0.832,0]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; IP_SCORE(-0.18)[ip: (0.04), ipnet: 18.220.0.0/14(0.19), asn: 16509(-1.10), country: US(-0.05)]; MIME_GOOD(-0.10)[text/plain]; DMARC_NA(0.00)[lonesome.com]; AUTH_NA(1.00)[]; RCPT_COUNT_FIVE(0.00)[6]; TO_MATCH_ENVRCPT_SOME(0.00)[]; RCVD_IN_DNSWL_MED(-0.20)[11.6.222.18.list.dnswl.org : 127.0.5.2]; NEURAL_SPAM_LONG(0.05)[0.049,0]; R_SPF_NA(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; R_DKIM_NA(0.00)[]; MIME_TRACE(0.00)[0:+]; ASN(0.00)[asn:16509, ipnet:18.220.0.0/14, country:US]; MID_RHS_MATCH_FROM(0.00)[]; RCVD_TLS_ALL(0.00)[]; RCVD_COUNT_TWO(0.00)[2] X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.29 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: Fri, 07 Feb 2020 01:05:59 -0000 On Tue, Feb 04, 2020 at 01:24:29PM +0100, Mathieu Arnold wrote: > This is why we have Tools/script/addport. Allow me to disagree by playing devil's advocate. (disclaimer: I am *not* picking on any one partiular committer here -- although, unfortunately, some of the people I've been sending private email to have taken it that way.) No, this is why we have *committers*. After I last posted last month about "please folks, test before commit", somewhere between 10 and 20 new ports have been added that simply. did. not. work. I'm *not* complaining that they did not work on powerpc64 -- that's a task that I picked up all by myself; it's my decision whether to do it or not. But: - "make patch" must at least work. This requires almost no time. This means the Makefile has to be sane, the files have to fetch+, all the files intended to be committed must be committed, and the patches must be right. (Suggestion: use "make distclean" then "make patch" to double-check fetching.) - "make describe" has to at least work. This requires no time. - Feeding the port to poudriere++ *must* not fail due to the metadata in /Makefile being missing or misspelled. Otherwise, someone (TM) has to make a commit to "fix bulk -a". (You may have noticed how many there are these days.) By skipping these simple tests, committers push work onto either current portmgrs or other people (including past portmgrs), leading to wasted time, resentment, and burnout. Committers *should* have already accepted that they are the gateway between raw patches and a working ports tree. Please, please, can we now get back to that? Look, I know we all make mistakes. But the things above result from people being in "too much of a hurry to test". Thanks. mcl + fetch, preferably from an original site, not just a FreeBSD.org echo ++ poudriere is what the FreeBSD package builders use. A commit that builds manually yet not under poudriere may be "technically" correct but users of packages won't think so.