From owner-freebsd-hackers@freebsd.org Sat Jun 2 06:48:25 2018 Return-Path: Delivered-To: freebsd-hackers@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 70AEAF7EBB6 for ; Sat, 2 Jun 2018 06:48:25 +0000 (UTC) (envelope-from ske@pkmab.se) Received: from mail1.bemta25.messagelabs.com (mail1.bemta25.messagelabs.com [195.245.230.67]) by mx1.freebsd.org (Postfix) with ESMTP id E64BE70437 for ; Sat, 2 Jun 2018 06:48:24 +0000 (UTC) (envelope-from ske@pkmab.se) Received: from [46.226.52.196] (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256 bits)) by server-3.bemta.az-b.eu-west-1.aws.symcld.net id 46/C6-23039-F0C321B5; Sat, 02 Jun 2018 06:41:19 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrGIsWRWlGSWpSXmKPExsVycM+PA7r8NkL RBm3ruCy2b/7H6MDoMePTfJYAxijWzLyk/IoE1oxNlx6wFPQrVryZfZipgfG7VBcjF4eQwClG ia+v3rFCOCeAnG2PmCCcaYwSZ5a1sXcxcnIIC5hLzGtrB0pwcIgIyEssOG8PEmYR0JU48/AAM 4jNLBAi8aF5HZjNKZAnceFjKyuILSGgIPFlxzwwm1dAUOLkzCcsEPU6Eu/6HoDVswloSMy71c 8CUWMqceDKJ0aI3mCJjls72CFsNYmr5zZB7dKWWLbwNfMERoFZSMbOQjJ2FpKyBYzMqxjNk4o y0zNKchMzc3QNDQx0DQ2NdA0tzXWNzPUSq3ST9FJLdctTi0t0DfUSy4v1iitzk3NS9PJSSzYx AsOXAQh2MK68kHyIUZKDSUmU91mVYLQQX1J+SmVGYnFGfFFpTmrxIUYZDg4lCV5ua6FoIcGi1 PTUirTMHGAkwaQlOHiURHi5QNK8xQWJucWZ6RCpU4yKUuK8fVZACQGQREZpHlwbLHovMcpKCf MyAh0ixFOQWpSbWYIq/4pRnINRSZj3MsgUnsy8Erjpr4AWMwEtfi0rALK4JBEhJdXA6Bt60fZ EkY4y/3muG9f55d4n9XqJef8XUn2/76a0+ZegmxuOLTgf1DVL/FXJ7KgD02vXu99fdm21c8uE gzfX1BU11cjvTDc/f/7z6Qli63/YG7tY+H2OnW50R9uw6UyCQ8+aOZyzrT5Ypq7yEXU7d6t17 +c+tu54WffJC/yXa2jt2ercUbSwQYmlOCPRUIu5qDgRAKkDXl/ZAgAA X-Env-Sender: ske@pkmab.se X-Msg-Ref: server-27.tower-284.messagelabs.com!1527921678!1499614!1 X-Originating-IP: [193.188.248.192] X-SYMC-ESS-Client-Auth: outbound-route-from=fail X-StarScan-Received: X-StarScan-Version: 9.9.15; banners=-,-,- X-VirusChecked: Checked Received: (qmail 3613 invoked from network); 2 Jun 2018 06:41:19 -0000 Received: from cdo-smtp31.composeit.net (HELO cdo-smtp31.composeit.net) (193.188.248.192) by server-27.tower-284.messagelabs.com with SMTP; 2 Jun 2018 06:41:19 -0000 Received: from cdo-smtp31.composeit.net (localhost [127.0.0.1]) by cdo-smtp31.composeit.net (Postfix) with ESMTP id B4A9C1D2E8 for ; Sat, 2 Jun 2018 08:41:18 +0200 (CEST) Received: from psy-app005.precio.lan (unknown [10.112.7.132]) by cdo-smtp31.composeit.net (Postfix) with ESMTP id 9DD411D2E3 for ; Sat, 2 Jun 2018 08:41:18 +0200 (CEST) Received: from berenice.precio.lan ([172.27.68.201]) by psy-app005.precio.lan with Microsoft SMTPSVC(8.5.9600.16384); Sat, 2 Jun 2018 08:41:18 +0200 Subject: Re: PRs are being closed for bogus reasons :-( To: freebsd-hackers@freebsd.org Date: Sat, 2 Jun 2018 08:41:16 +0200 (CETDST) Cc: imp@bsdimp.com, dieterbsd@gmail.com, phk@phk.freebsd.dk, kristof@sigsegv.be In-Reply-To: from "Kristof Provost" at Jun 1, 18 05:27:28 pm X-Mailer: ELM [version 2.4 PL23] MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 From: Kristoffer Eriksson Message-ID: <201806020841.aa01298@berenice.pkmab.se> X-OriginalArrivalTime: 02 Jun 2018 06:41:18.0244 (UTC) FILETIME=[B601F640:01D3FA3C] X-Virus-Scanned: ClamAV using ClamSMTP Content-Transfer-Encoding: quoted-printable X-Mailman-Approved-At: Sat, 02 Jun 2018 11:19:40 +0000 X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 02 Jun 2018 06:48:25 -0000 On Jun 1, 2018 at 17:27, Kristof Provost wrote: > On 1 Jun 2018, at 17:09, Warner Losh wrote: >> Well, not quite true. I've had several people send me pointers to >> bugs over the years and engage me when I tell them that the patch >> isn't quite right. >> That conversation is easier, to my mind, in Phabricator, though. >> There's no substitute for making good connections and motivating >> volunteers to want to help you. That gives much better results >> than filing and forgetting and hoping for the best. As a committer, >> I find it a low return on investment to go looking at random PRs. >> I find it a much higher return on investment when I have a history >> with someone (even a short one). ... > For better or worse, the fact is that both patches and bug reports > fare better if their submitter actively advocates for them. > > I don't mean to suggest that it is somehow the fault of the submitter > if bugs don't get fixed. Instead I want to point at this as something > people can do to help, even if they don't have commit access, or even > if they don't know how to read or write code. As a submitter of a few bug reports in bugzilla, it would have been extremely useful to me if there had been some hints somewhere in the bug report submission process about what I could do to follow up and promote that report (or fix) to relevant people or forums, in stead of only finding that out much later when asking on the mailing list for any response. As a first-time submitter, you naturally assume that after submitting a good detailed bug report, your job is done, and that there is really nothing more you can do (for those who are not able to fix the bug themselves). Nothing in bugzilla gives any reason to think otherwise (as far as I remember). And that is very misleading. Maybe(?) there is some information about that somewhere else, but you really can't read absolutely everything there is in the whole project before submitting a bug report. - Also, for those who are able to also submit patches together with their bug reports, maybe more could be done to guide them on how to improve their patches on the way to a more committable state? But preferrably not in a way that discourages people from submitting anything at all. I would guess that many times, a patch submitted by a bug reporter may just be meant to serve as a proof or example that the bug is real and goes away if certain changes are made to the source code, and not necessarily as a production quality committable fix. Or anything in between those two. Even then, having a patch should be much better than having nothing, for the maintainer (or who-ever tries to fix the bug) to work with, but not necessarily meant to be committed as-is. Maybe it would be helpful if the submitter would be asked to self- grade what they think their patch is good for, on some kind of scale? Or indicate what steps they have already done (or not done) themselves in the process from plain bug report to submittable patch, if a couple of such steps could be suggested. - Another question that occurs to me: Who is it that is supposed to take an interrest in the bug reports? Is just any random volunteer supposed to come by, who doesn't know anything, and look at them if they feel like it? And then be able to fix them? That doesn't seem very likely to happen. Maybe occasionally, but not sustainably. I would think that if there was a bug report relating to some code created or maintained by a specific person or group of persons or some upstreams project, then the chances would increase significantly if the report was sent raight away to that group of people. For instance, if I would have created some small part of the FreeBSD software, or ported some software to it, or if some other woftware that I created would be ported to FreeBSD, then I would like to receive any bug reports relating to that specific part. (Personally I would even be interrested if there were some bug reports in the same parts of code that I have submitted patches for before, which I have indeed done.) Does that not happen in bugzilla? Doesn't bugzilla have any information about who maintains or who created various parts of the project? Regards/Kristoffer Eriksson