On Tue 2014-07-22 08:27:22, Kalle Valo wrote:
> Borislav Petkov writes:
>
> > No, checkpatch should only check patches and not whole file. It is
> > wrong. Fullstop.
>
> Please don't remove --file, I use it all the time when maintaining
> ath6kl and ath10k. It's a lot easier in my workflow to te
Borislav Petkov writes:
> No, checkpatch should only check patches and not whole file. It is
> wrong. Fullstop.
Please don't remove --file, I use it all the time when maintaining
ath6kl and ath10k. It's a lot easier in my workflow to test the whole
driver in one go than start testing individual
On Fri, Jul 18, 2014 at 07:57:13AM -0700, Joe Perches wrote:
> I think your viewpoint is a bit tainted as you're the
> same guy that wrote "fuck readability".
>
> https://lkml.org/lkml/2013/11/19/116
It seems you still haven't understood what I actually meant.
If you'd tried to understand what I
On Fri, 2014-07-18 at 16:49 +0200, Borislav Petkov wrote:
> On Fri, Jul 18, 2014 at 07:35:26AM -0700, Joe Perches wrote:
> > Using the one-liner above also makes it harder to
> > automate checkpatch neatening and avoid using Lindent:
>
> automated checkpatch?? More idiocy... we want less, in case
On Fri, Jul 18, 2014 at 07:43:57AM -0700, Guenter Roeck wrote:
> Thinking about it, Linux itself can be abused as well. We should
> take it away to prevent that abuse from happening.
Whatever floats your boat, dude.
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is
On Fri, Jul 18, 2014 at 07:35:26AM -0700, Joe Perches wrote:
> Using the one-liner above also makes it harder to
> automate checkpatch neatening and avoid using Lindent:
automated checkpatch?? More idiocy... we want less, in case you've
forgotten.
Let me state it again:
The thing is called che
On 07/18/2014 07:27 AM, Borislav Petkov wrote:
On Fri, Jul 18, 2014 at 07:21:14AM -0700, Christoph Hellwig wrote:
Or he could just use the existing -f flag.
If you mean the -f flag to checkpatch, he's removing them both.
Two different 'he'.
I really don't understand why you're trying to ed
On Fri, 2014-07-18 at 16:24 +0200, Borislav Petkov wrote:
> On Fri, Jul 18, 2014 at 04:17:42PM +0200, Lars-Peter Clausen wrote:
> > --file is a in my opinion useful option and at least I use it on a
> > regular basis.
[]
> diff -urN /dev/null $file | ./scripts/checkpatch.pl -
Using the one-liner a
On Fri, Jul 18, 2014 at 07:21:14AM -0700, Christoph Hellwig wrote:
> Or he could just use the existing -f flag.
If you mean the -f flag to checkpatch, he's removing them both.
> I really don't understand why you're trying to educate people by
> taking their toy away. They'll find another one in n
On Fri, Jul 18, 2014 at 04:17:42PM +0200, Lars-Peter Clausen wrote:
> --file is a in my opinion useful option and at least I use it on a
> regular basis.
Use
diff -urN /dev/null $file | ./scripts/checkpatch.pl -
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is fi
On Fri, Jul 18, 2014 at 06:56:12AM -0700, Guenter Roeck wrote:
> Yes, that came up before. Or I can revert the patch locally, or keep
> an old version of checkpatch around. You make me suffer
Come on, you suffer from a one-liner?! Script it or whatever. Puh-lease!
The thing is inviting lazy wanke
On Fri, Jul 18, 2014 at 03:46:29PM +0200, Richard Weinberger wrote:
> As capable kernel hacker you can still use a command like:
> diff -urN /dev/null $file | ./scripts/checkpatch.pl -
Or he could just use the existing -f flag. I really don't understand
why you're trying to educate people by taki
On 07/17/2014 05:34 PM, Richard Weinberger wrote:
[...]
In order to deal with that bad habit let's remove the --file option
and bring checkpatch.pl back to its original purpose.
I don't think this is a good solution the problem and I'm not sure how
successful it will actually be at fixing the
On Fri, Jul 18, 2014 at 03:46:29PM +0200, Richard Weinberger wrote:
> Am 18.07.2014 15:37, schrieb Guenter Roeck:
> > On 07/18/2014 01:23 AM, Borislav Petkov wrote:
> >> On Fri, Jul 18, 2014 at 12:29:37AM -0700, Guenter Roeck wrote:
> >>> First, 'abuse' is a relative term. It describes a use you
>
Am 18.07.2014 15:37, schrieb Guenter Roeck:
> On 07/18/2014 01:23 AM, Borislav Petkov wrote:
>> On Fri, Jul 18, 2014 at 12:29:37AM -0700, Guenter Roeck wrote:
>>> First, 'abuse' is a relative term. It describes a use you
>>> (and possibly many others) may find objectionable, but that
>>> does not m
On 07/18/2014 01:23 AM, Borislav Petkov wrote:
On Fri, Jul 18, 2014 at 12:29:37AM -0700, Guenter Roeck wrote:
First, 'abuse' is a relative term. It describes a use you
(and possibly many others) may find objectionable, but that
does not mean all uses are objectionable.
Do you actually have a v
On Fri, Jul 18, 2014 at 12:29:37AM -0700, Guenter Roeck wrote:
> First, 'abuse' is a relative term. It describes a use you
> (and possibly many others) may find objectionable, but that
> does not mean all uses are objectionable.
Do you actually have a valid use case for keeping the cmdline switch
On 07/17/2014 08:34 AM, Richard Weinberger wrote:
checkpatch.pl is a nice tool to find issues in patches.
Sadly this tool gets more and more abused by various people to create
style cleanups for source files within the kernel.
In order to deal with that bad habit let's remove the --file option
a
On Thu, 17 Jul 2014, Joe Perches wrote:
> On Thu, 2014-07-17 at 17:34 +0200, Richard Weinberger wrote:
> > checkpatch.pl is a nice tool to find issues in patches.
>
> grep is a similar tool, just less automated.
>
> > Sadly this tool gets more and more abused by various people to create
> > styl
On Thu, 2014-07-17 at 18:02 +0200, Borislav Petkov wrote:
> On Thu, Jul 17, 2014 at 08:51:23AM -0700, Joe Perches wrote:
> > grep is a similar tool, just less automated.
[]
> checkpatch should only check patches and not whole file.
> It is wrong. Fullstop.
No worries, we just disagree about the
On Thu, Jul 17, 2014 at 08:51:23AM -0700, Joe Perches wrote:
> grep is a similar tool, just less automated.
And still, we don't want to encourage people with useless cleanups.
> I think it should really only be used in --file mode
> on drivers/staging/.
Also dumb idea - having to look at a singl
On Thu, 2014-07-17 at 17:34 +0200, Richard Weinberger wrote:
> checkpatch.pl is a nice tool to find issues in patches.
grep is a similar tool, just less automated.
> Sadly this tool gets more and more abused by various people to create
> style cleanups for source files within the kernel.
> In ord
On Thu, Jul 17, 2014 at 05:34:28PM +0200, Richard Weinberger wrote:
> checkpatch.pl is a nice tool to find issues in patches.
> Sadly this tool gets more and more abused by various people to create
> style cleanups for source files within the kernel.
> In order to deal with that bad habit let's re
23 matches
Mail list logo
