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


Groups > linux.kernel > #1483525

Re: [PATCH] checkpatch: Minimize checkpatch induced patches...

From Christian Borntraeger <borntraeger@de.ibm.com>
Newsgroups linux.kernel
Subject Re: [PATCH] checkpatch: Minimize checkpatch induced patches...
Date 2016-09-14 20:20 +0200
Message-ID <shmWR-4ek-13@gated-at.bofh.it> (permalink)
References <shmDw-3Sa-13@gated-at.bofh.it> <shmDw-3Sa-21@gated-at.bofh.it> <shmNc-4aI-31@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 09/14/2016 08:06 PM, Joe Perches wrote:
> On Wed, 2016-09-14 at 19:56 +0200, Christian Borntraeger wrote:
> 
>> This will certainly help to reduce the noise. On the other hand I remember Linus
>> saying something along the line that he does not like the -f parameter (and he
>> prefers to set this automatically). So while I like the approach I am not happy
>> enough to ack right now - still looking for a better alternative :-/
> 
> Linus likely hasn't used checkpatch in a decade or so.
> 
> Taste and judgment can't be scripted anyway.
> 
> Let me know if you find an alternative.

You know what. 
with some additional writing like 
"Existing code outside staging is not supposed to be "fixed" to match checkpatch.
Please do not send checkpatch initiated patches for those files"
near the newly created warn

Acked-by: Christian Borntraeger <borntraeger@de.ibm.com>

Feel free to improve my sentence to something proper.

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


Thread

Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Christian Borntraeger <borntraeger@de.ibm.com> - 2016-09-14 20:00 +0200
  Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Joe Perches <joe@perches.com> - 2016-09-14 20:10 +0200
    Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Christian Borntraeger <borntraeger@de.ibm.com> - 2016-09-14 20:20 +0200
      Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Christian Borntraeger <borntraeger@de.ibm.com> - 2016-09-14 20:30 +0200
      Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Joe Perches <joe@perches.com> - 2016-09-14 20:30 +0200
      Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Greg KH <greg@kroah.com> - 2016-09-14 20:40 +0200
        Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Christian Borntraeger <borntraeger@de.ibm.com> - 2016-09-14 21:00 +0200
          Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Joe Perches <joe@perches.com> - 2016-09-14 21:20 +0200
        Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Christian Borntraeger <borntraeger@de.ibm.com> - 2016-09-18 21:40 +0200
  Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Josh Triplett <josh@joshtriplett.org> - 2016-09-15 02:00 +0200
    Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Josh Triplett <josh@joshtriplett.org> - 2016-09-15 02:10 +0200
    Re: [PATCH] checkpatch: Minimize checkpatch induced patches... Joe Perches <joe@perches.com> - 2016-09-15 02:10 +0200

csiph-web