public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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.

      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