Date: Tue, 25 Jun 2019 12:03:45 +0200 From: =?UTF-8?B?VMSzbA==?= Coosemans <tijl@FreeBSD.org> To: Kubilay Kocak <koobs@FreeBSD.org> Cc: Piotr Kubaj <pkubaj@anongoth.pl>, "Tobias C. Berner" <tcberner@gmail.com>, ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r505045 - head/sysutils/plasma5-ksysguard Message-ID: <20190625120345.0a2cc2eb@kalimero.tijl.coosemans.org> In-Reply-To: <5878f408-2030-7f57-ec1e-5f45e814433f@FreeBSD.org> References: <201906241810.x5OIAu1h080487@repo.freebsd.org> <CAOshKtcPHHa4%2Bv2kL_aNKXzoXs1VkGw0nEAx3PkaArPJ6kCGzw@mail.gmail.com> <20190624194627.GB49520@ThinkPad-X200.g.anongoth.pl> <CAOshKtegUmUYfdnDNmt9wuk1cSC_z_qpz8td597zC4y3Dup_-w@mail.gmail.com> <20190624202703.GA68048@ThinkPad-X200.g.anongoth.pl> <8eab69dc-52bb-a187-6a30-565ae58f4512@FreeBSD.org> <20190625082911.GA63640@KGPE-D16> <5878f408-2030-7f57-ec1e-5f45e814433f@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 25 Jun 2019 18:45:33 +1000 Kubilay Kocak <koobs@FreeBSD.org> wrote: > On 25/06/2019 6:29 pm, Piotr Kubaj wrote: >> To be honest, I fail to see the meaning of this flag. >> >> If it's not about approval, then what does this flag actually mean? Only >> that "I acknowledge that there's a problem"? > > It means feedback is required. Feedback can take many forms. Not all > bugs are patch submissions requiring (only) approval from a maintainer. > > Take for example, a bug report without a patch. maintainer-feedback? is > set when issue is created. The maintainer comes back with 'i can > reproduce the problem' and sets maintainer-feedback + (feedback > provided). Triage sets need-patch keyword requesting a patch to fix the > issue and sets maintainer-feedback? again, feedback this time being in > the form of a patch. > >> Then maybe work-in-progress? As in, the maintainer is working on the fix. > > This doesn't cover feedback of forms that don't involve work/patches, > the vast majority, and this is already covered by needs-patch keyword in > any case. > > Again, if there's any way to improve the maintainer-feedback flag name > to not mean 'approval' (as thats not what its for), I'd been keen to > hear ideas. But what purpose does it serve? Who's workflow depends on the flag and why? It all seems like pointless paperwork to me. When maintainer feedback is necessary seems obvious: if the last comment isn't his, his feedback is needed. Why do we have to set a flag for that?
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190625120345.0a2cc2eb>