Sure. But the best a non-comitter can do is to supply a patch tested against HEAD. If the patch rots because it hasn’t been committed six months down the line it’s not my fault.
--
Bob Bishop
r...@gid.co.uk
>
> > On 1 Jun 2018, at 15:41, Warner Losh <i...@bsdimp.com> wrote:
> >
> >> On Fri, Jun 1, 2018 at 8:10 AM, Bob Bishop <r...@gid.co.uk> wrote:
> >> Hi,
> >>
> >> > On 31 May 2018, at 22:14, Poul-Henning Kamp <p...@phk.freebsd.dk>
> wrote:
> >> >
> >> > --------
> >> > In message <CANCZdfovyg61=b0-60ZD426Z=774Wxm9hJxZ=bSqReW_fD4DoA@
> mail.gmail.com>
> >> > , Warner Losh writes:
> >> >
> >> >> There's a problem with the PR database: there's too many bugs.
> >> >
> >> > And despite the valiant efforts of a number of people over the
> >> > lifetime of the project, it has always had so many bugs that
> >> > everybody just threw their hands in the air and walked away.
> >> >
> >> > The way to improve the situation is to fix PR's, not to complain
> >> > about PRs.
> >>
> >> Indeed. But look at the number of PRs with patches that are stuck in
> that state. Not pretty.
> >
> > Over the years I've committed dozens of PRs that had patches in them.
> The sad truth is that only about 10-15% of them have comitable patches in
> them when submitted. And that number decays over time as things age in
> bugzilla. [etc]
>
> Sure. But the best a non-comitter can do is to supply a patch tested
> against HEAD. If the patch rots because it hasn’t been committed six months
> down the line it’s not my fault.
>
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).
Fixing this broken state of affairs is not going to be easy...
Warner
The problem isn't bitrot, the problem is that many patches amount to
"here's a hack that works for me," and that isn't necessarily
committable. A committer typically has to do almost as much work to
figure out whether the patch is appropriate for all users on all arches
as they would have to do to develop a fix from scratch. Even if the
submitter has mad skills and submits a perfect patch, better than what
the committer would have done from scratch, the work to analyze
everything and decide whether that's the case still has to be done.
-- Ian
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.
Regards,
Kristof
It was indeed not IPv6 ready...
Jan
ManiaC++
Jan Knepper
Hi Andrey,
Maybe this proposal was made in jest, but I actually like the idea.
The dominant noise of freebsd-bugs@ comes from follow-up comments, bug
status notifications (sometimes bulk changes made by e.g., Eitan), or
direct email reply discussion (not really sure why bugs@ isn't just
treated as announce-only).
It's still sort of a firehose if you *just* receive new bug reports,
but it's much more manageable and you can click through any that look
interesting and mark the rest read with no risk of future
notification.
So my modified proposal is:
1. Create an announce-only bugs list that only receives new ports.
Maybe it can have a name incorporating "triage." I don't care.
2. Subscribe committers to it by default.
3. Encourage people to stay subscribed and help them set up inbox
filters to separate it from higher priority email, so it's less of a
nuisance.
4. Finally, allow opting out. Bug triage isn't for everyone. But it
is definitely an area where "many hands make light work," and we don't
have a lot of hands doing it right now.
Special thanks to Mark, who spends an amazing amount of time helping
to triage the incoming bug firehose, annotate bugs with patches, and
get bugs in front of relevant eyeballs.
Best,
Conrad
What Is Core Doing About It?™
Jan
ManiaC++
Jan Knepper
I like the idea of a list that just annouces new bugs but contains no
other traffic. I sometimes stumble across bugs by accident that I feel
like are in my wheelhouse or are trivial to fix. An announce-only list
would probably make a few more of those drop into my lap.
Do you envision people being able to comment/reply/post to the list in
general? What I'm curious about is the level of non-announce mail
that's going to be on the list. If it turns into general chit-chat
about the bugs that are announced, the noise level goes way way up.
Also, that would encourage discussion related to the bugs which should
probably happen in bugzilla comments rather than out-of-band mail.
Hmm, something that could reduce the traffic even more would be to send
out a once-daily mail summarizing the short description lines of all
bugs entered in the past 24 hours. Maybe that could be done and sent to
a few appropriate existing lists (one or more of stable@, current@,
ports@, etc, depending on the metadata in the PRs).
-- Ian
On Fri, Jun 1, 2018 at 11:40 AM, Ian Lepore <i...@freebsd.org> wrote:
> On Fri, 2018-06-01 at 11:23 -0700, Conrad Meyer wrote:
>> ...
>> So my modified proposal is:
>>
>> 1. Create an announce-only bugs list that only receives new ports.
Sorry, typo: that should be "new reports."
>> ...
>
> Do you envision people being able to comment/reply/post to the list in
> general?
Nope. I think it should be Just New Bugs to keep noise down. The
idea is to get the summary and description in front of eyeballs, and
no follow-up notification unless you explicitly CC yourself. This is
basically the model I use today, except I had to set up explicit email
filters to get it from the existing list, and that's a totally
unnecessary burden to put on everyone.
As Ben Kaduk says, discussion can and should happen in Bugzilla.
> What I'm curious about is the level of non-announce mail
> that's going to be on the list. If it turns into general chit-chat
> about the bugs that are announced, the noise level goes way way up.
> Also, that would encourage discussion related to the bugs which should
> probably happen in bugzilla comments rather than out-of-band mail.
Totally agree. It's most useful announce-only.
> Hmm, something that could reduce the traffic even more would be to send
> out a once-daily mail summarizing the short description lines of all
> bugs entered in the past 24 hours. Maybe that could be done and sent to
> a few appropriate existing lists (one or more of stable@, current@,
> ports@, etc, depending on the metadata in the PRs).
I prefer the other model, but I believe mailman can accommodate this
mode simultaneously with a single list — I think users can configured
it to do this on a individual user basis.
Conrad