Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1539879

Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move

From Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Newsgroups linux.kernel
Subject Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move
Date 2016-12-10 19:20 +0100
Message-ID <sMUpz-5Az-9@gated-at.bofh.it> (permalink)
References <sMUfT-5x2-3@gated-at.bofh.it> <sMUfT-5x2-15@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw

On Sat, Dec 10, 2016 at 01:05:59PM -0500, James Simmons wrote:
> From: Ben Evans <bevans@cray.com>
> 
> It was found if you sort the headers alphabetically
> that it reduced patch conflicts. This patch sorts
> the headers alphabetically and also place linux
> header first, then uapi header and finally the
> lustre kernel headers.

I still don't agree, when did you last have a patch conflict with this
file in the .h section?  And exactly how hard was it to fix it up?

I'm all for cleanups, but really, this is useless.  And I said so the
last time you sent it...

greg k-h

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 3/5] staging: lustre: headers: sort headers affected by obdo move James Simmons <jsimmons@infradead.org> - 2016-12-10 19:10 +0100
  Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected  by obdo move Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-12-10 19:20 +0100
    Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by  obdo move Ben Evans <bevans@cray.com> - 2016-12-12 16:00 +0100
      Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected  by obdo move Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-12-12 17:40 +0100
        Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected  by obdo move Joe Perches <joe@perches.com> - 2016-12-12 19:20 +0100
          Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected  by obdo move James Simmons <jsimmons@infradead.org> - 2016-12-12 20:50 +0100
            Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected  by obdo move Joe Perches <joe@perches.com> - 2016-12-12 21:00 +0100
        Re: [PATCH v2 3/5] staging: lustre: headers: sort headers affected by  obdo move Ben Evans <bevans@cray.com> - 2016-12-12 19:30 +0100

csiph-web