From: Michael Witten <mfwitten@gmail.com>
To: Rob Landley <rob@landley.net>
Cc: Jiri Kosina <trivial@kernel.org>,
Randy Dunlap <rdunlap@infradead.org>,
David Airlie <airlied@linux.ie>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 trivial 0/7] Miscellaneous Trivialities
Date: Wed, 21 Aug 2013 17:24:00 -0000 [thread overview]
Message-ID: <f8967ec424bc4211974dea8ebdf5b3f5-mfwitten@gmail.com> (raw)
In-Reply-To: <1377068657.2737.90@driftwood>
On Wed, 21 Aug 2013 02:04:17 -0500, Rob Landley wrote:
> On 08/20/2013 10:32:02 PM, Michael Witten wrote:
>
>> On Tue, 20 Aug 2013 19:19:37 -0500, Rob Landley wrote:
>>
>>> Hence asking if we really needed
>>> three separate commits to accomplish something that didn't actually
>>> need to be done in the first place.)
>>> ...
>>> Actually my objection is that it's not worth the churn in the commit logs.
>>
>> Naturally, we don't NEED three separate commits! Squash all of them
>> into one commit if that's something worth hissing about.
>>
>> Do you need help with the relevant commands?
>
> The correct response to someone disagreeing you is to patronize them?
That's rich.
> Query: did you notice the phrase "something that didn't actually need
> to be done in the first place"? You quoted it and everything, so I'm
> assuming you ignored it intentionally rather than simply not noticing.
I quote my quote of your text:
> Hence asking if we really needed three separate commits to accomplish
> something that didn't actually need to be done in the first place.)
> ...
> Actually my objection is that it's not worth the churn in the commit
> logs.
Query: Do you not even know what you write, documentation maintainer?
>> Because you are the former and current maintainer of large and active
>> projects, I'd expect you to *appreciate* the value of taking in
>> fine-grained patches for REVIEW (even if you don't appreciate their
>> sum total).
>
> I reviewed them. The result of that review is what you're objecting to.
I'm objecting to your style.
> You misunderstand the nature of the position: this is volunteer
> janitorial work. I don't get paid for it
Some janitors mop whole floors. Other janitors scrub patches of moldy
grout with a toothbrush. There's no reason for one to belittle the
other---especially when *both* aren't getting paid.
> (Your indignant and self-important reaction to that evaluation _has_ changed
> my opinion of your technical judgement and likelihood of listening to
> you in the future, so that's something.)
My sentiments exactly.
You are a master of irony.
> Nothing stops one of the other maintainers from taking your patch over
> my objections. Most documentation goes in through other trees anyway,
> generally as part of series with code and documentation components. But
> _my_ evaluation remains "NAK".
Good grief. The destiny of these trivial patches is irrelevant at this
point in our conversation; please reflect instead on the destiny of
your relationship with other people.
prev parent reply other threads:[~2013-08-21 17:28 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-20 16:02 [PATCH v3 trivial 0/7] Miscellaneous Trivialities Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 1/7] Docs: Kconfig: For readability, offset modifiers with commas Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 2/7] Docs: Kconfig: Use consistent whitespace indentation Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 3/7] Docs: Kconfig: Clean up the radiotap documentation Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 4/7] Docs: Kconfig: `devlopers' -> `developers' Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 5/7] DRM: comment: `halve' -> `half' Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 6/7] DRM: comment: `gdm_proc_lists' -> `drm_info_lists' Michael Witten
2013-08-20 16:05 ` [PATCH v3 trivial 7/7] DRM: cleanup: Remove unused `gamma_size' Michael Witten
2013-08-20 21:20 ` [PATCH v3 trivial 0/7] Miscellaneous Trivialities Rob Landley
2013-08-20 22:27 ` Michael Witten
2013-08-21 0:19 ` Rob Landley
2013-08-21 3:32 ` Michael Witten
2013-08-21 7:04 ` Rob Landley
2013-08-21 17:24 ` Michael Witten [this message]
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=f8967ec424bc4211974dea8ebdf5b3f5-mfwitten@gmail.com \
--to=mfwitten@gmail.com \
--cc=airlied@linux.ie \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@infradead.org \
--cc=rob@landley.net \
--cc=trivial@kernel.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