From: xerofoify@gmail.com (nick)
To: kernelnewbies@lists.kernelnewbies.org
Subject: Filtering noise is about protecting resourses
Date: Tue, 09 Dec 2014 06:33:07 -0500 [thread overview]
Message-ID: <5486DDF3.6090401@gmail.com> (raw)
In-Reply-To: <87388pch0a.fsf_-_@nemi.mork.no>
Bjorn,
I understand that now after reading your message. To be honest, I started out like this because I had no idea,
where to start. If your willing to give me a place to start, that is of use I will be glad to help out. Over
time, I hope we can work this out.
Nick
On 2014-12-09 04:24 AM, Bj?rn Mork wrote:
> Philipp Muhoray <philipp.muhoray@gmail.com> writes:
>
>> Not that I have any say in this, but I feel like a ban should rather
>> be justified by someone's behavior instead of incorrect patches.
>
> It's not a ban, it's a protective filter. Maintainers and reviewers are
> limited resources. We should not waste them.
>
>> I
>> guess most of us have send awful patches at some point, the question
>> though is how we dealt with it. I'm not saying the ban should be
>> lifted, I'm just saying we should communicate the right arguments for
>> his ban (instead of blaming him for commit messages he didn't even
>> write).
>
> If you look at what actually happened, you'll see a very good example of
> why the filter is still required: The original patch was yet another
> completely pointless fixme-comment deletion, without any real
> explanation whatsoever in the commit message. And it wasn't even
> properly formatted with a subsystem prefixed subject etc. So the
> maintainer had to spend time trying to fix up the commit message and
> figuring out why it was OK to delete those fixme comments. As has been
> pointed out here, that explanation could still be incomplete. I guess
> the maintainer didn't want to spend hours looking at something as
> pointless as this. The problem is that he didn't realize that this
> patch was a waste of time before spending time on it at all.
>
> Which is exactly why the maintainers should be protected against having
> to look at stuff like this, if possible. And in this case it *is*.
> There are exactly zero examples of valuable patches from that author.
> If you look at the history of accepted patches, you will find that in
> each and every case there is some reviewer or maintainer doing the
> *real* work - figuring out that the patch is OK and explaining why. And
> the result is still patches without any real value. They don't solve
> any problem for anyone. They are the result of stupid and mindless
> grepping for a specific word in comments.
>
> Yes, we have all wasted time for maintainers and reviewers by sending
> them stuff we shouldn't have. That's part of the game. The problem in
> this case is the massive distribution over an insane number of
> subsystems in combination with the inability to learn anything at all.
> Wasting one maintainer's time once is excusable. Wasting hundreds of
> maintainer's time over and over again is absolutely not. It's
> potentionally destructive to the whole project if it is allowed to
> continue.
>
> This very thread is yet another example of the contentless noise from
> this source, and I hate myself for having wasted your time having to
> read this. Sorry about that.
>
> But I write it in the hope that you will understand that the filtering
> is *not* about punishing anyone. It is about protecting or most valuable
> resources.
>
> And if anyone still wonders: Requests for "ban removal" has no value to
> the community, and are therefore the exact opposite of what's required
> to have the filter removed.
>
>
> Bj?rn
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
>
next prev parent reply other threads:[~2014-12-09 11:33 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-08 3:15 Remove Ban? nick
2014-12-08 15:32 ` Valdis.Kletnieks at vt.edu
2014-12-08 15:56 ` Nick Krause
2014-12-09 6:50 ` Avinash Patil
2014-12-09 7:56 ` Philipp Muhoray
2014-12-09 8:05 ` Sudip Mukherjee
2014-12-09 9:24 ` Filtering noise is about protecting resourses (was: Re: Remove Ban?) Bjørn Mork
2014-12-09 11:33 ` nick [this message]
2014-12-09 18:27 ` Filtering noise is about protecting resourses Valdis.Kletnieks at vt.edu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5486DDF3.6090401@gmail.com \
--to=xerofoify@gmail.com \
--cc=kernelnewbies@lists.kernelnewbies.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).