From: David Miller <davem@davemloft.net>
To: william.allen.simpson@gmail.com
Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: warning: massive change to conditional coding style in net?
Date: Mon, 30 Nov 2009 12:36:44 -0800 (PST) [thread overview]
Message-ID: <20091130.123644.203668577.davem@davemloft.net> (raw)
In-Reply-To: <4B13A025.7000103@gmail.com>
From: William Allen Simpson <william.allen.simpson@gmail.com>
Date: Mon, 30 Nov 2009 05:36:21 -0500
> Over the past several days, David Miller (with help from Joe Perches)
> made sweeping changes to the format of conditional statements in the
> net tree -- the equivalent of mass patches that change spaces.
>
> This makes writing patches for multiple versions of the tree very
> difficult, and will make future pullups problematic. It's enough to
> make a grown man cry.... Patching conflicts everywhere!
William, you're unreasonable.
We asked you to follow a certain style, and then you immediately
complain that the style isn't followed consistently in the tree, and
therefore as a consequence you shouldn't be required to follow it.
Then Joe comes and submits patches making the tree follow the style
more consistently. See, instead of merely complaining like you did,
he proactively did something positive.
Now you're complaining because this makes your patches harder to
maintain.
You're being difficult about this from every possible angle. I can
only conclude that for whatever reason you don't want to have any
requirements made upon you for your submission. You've handled this
negatively in every way possible, and every step of the way. No
matter how hard other people have been to try and help you, you've
continued to do this.
Whereas if you had merely fixed up the coding style as I and others
have asked you, your code would be in my tree weeks ago.
Why is it such a huge deal to follow the coding style that the
maintainer (and other prominent developers of that subsystem) are
asking you to follow?
next prev parent reply other threads:[~2009-11-30 20:36 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-30 10:36 warning: massive change to conditional coding style in net? William Allen Simpson
2009-11-30 13:44 ` Jarek Poplawski
2009-11-30 13:54 ` Alan Cox
2009-11-30 19:39 ` Jarek Poplawski
2009-11-30 17:56 ` Joe Perches
2009-12-01 16:08 ` William Allen Simpson
2009-12-01 16:49 ` Eric Dumazet
2009-12-01 17:43 ` Jarek Poplawski
2009-11-30 20:36 ` David Miller [this message]
2009-12-01 17:56 ` William Allen Simpson
2009-12-01 18:30 ` Eric Dumazet
2009-12-01 23:28 ` David Miller
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=20091130.123644.203668577.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=william.allen.simpson@gmail.com \
/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).