Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 12:16 PM, Stefan Richter wrote: Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - If somebody is hacking kernel, I think

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Stefan Richter
Daniel Walker wrote: > On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: >> >> On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: >> >>> git show 9914cad54c79d0b89b1f066c0894f00e1344131c >> >> | ./scripts/checkpatch.pl - >> If somebody is hacking kernel, I think he should know the - trick

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 01:47 -0800, Daniel Walker wrote: > On Fri, 2008-01-11 at 10:41 +0100, Jiri Slaby wrote: > > On 01/11/2008 10:36 AM, Daniel Walker wrote: > > > On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: > > >> If somebody is hacking kernel, I think he should know the - trick used

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 01:30 -0800, Daniel Walker wrote: > On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: > > On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: > > > On 01/11/2008 10:17 AM, Daniel Walker wrote: > > > > On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: > > > >> On

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:41 +0100, Jiri Slaby wrote: > On 01/11/2008 10:36 AM, Daniel Walker wrote: > > On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: > >> If somebody is hacking kernel, I think he should know the - trick used in > >> many > >> programs, but do not consider this as a

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:36 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: If somebody is hacking kernel, I think he should know the - trick used in many programs, but do not consider this as a nack. I'm hacking the kernel, and I didn't know the - trick .. So you have

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: > On 01/11/2008 10:30 AM, Daniel Walker wrote: > > On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: > >> On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: > >>> git show 9914cad54c79d0b89b1f066c0894f00e1344131c > >> |

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:30 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - JftSoC: git show

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: > On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: > > On 01/11/2008 10:17 AM, Daniel Walker wrote: > > > On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: > > >> On 01/11/2008 05:10 AM, Daniel Walker wrote: > > >>> A little

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: > On 01/11/2008 10:17 AM, Daniel Walker wrote: > > On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: > >> On 01/11/2008 05:10 AM, Daniel Walker wrote: > >>> A little feature addition to allow checkpatch.pl to check patches piped > >>> into

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:17 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. You can still add - as an

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: > On 01/11/2008 05:10 AM, Daniel Walker wrote: > > A little feature addition to allow checkpatch.pl to check patches piped > > into it, in addition to specific file arguments. > > You can still add - as an argument to check stdin. In which way

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. You can still add - as an argument to check stdin. In which way is this better? regards, -- Jiri Slaby Faculty of Informatics,

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. You can still add - as an argument to check stdin. In which way is this better? regards, -- Jiri Slaby Faculty of Informatics,

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. You can still add - as an argument to check stdin. In which way is

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:17 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. You can still add - as an

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: On 01/11/2008 10:17 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to allow checkpatch.pl to check patches piped into it, in addition

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: On 01/11/2008 10:17 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008 05:10 AM, Daniel Walker wrote: A little feature addition to

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Stefan Richter
Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - If somebody is hacking kernel, I think he should know the - trick used in many

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 01:47 -0800, Daniel Walker wrote: On Fri, 2008-01-11 at 10:41 +0100, Jiri Slaby wrote: On 01/11/2008 10:36 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: If somebody is hacking kernel, I think he should know the - trick used in many

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:36 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: If somebody is hacking kernel, I think he should know the - trick used in many programs, but do not consider this as a nack. I'm hacking the kernel, and I didn't know the - trick .. So you have

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: On 01/11/2008 10:30 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl -

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 10:30 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - JftSoC: git show

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Daniel Walker
On Fri, 2008-01-11 at 10:41 +0100, Jiri Slaby wrote: On 01/11/2008 10:36 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: If somebody is hacking kernel, I think he should know the - trick used in many programs, but do not consider this as a nack. I'm

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Bernd Petrovitsch
On Fre, 2008-01-11 at 01:30 -0800, Daniel Walker wrote: On Fri, 2008-01-11 at 10:23 +0100, Bernd Petrovitsch wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: On 01/11/2008 10:17 AM, Daniel Walker wrote: On Fri, 2008-01-11 at 09:52 +0100, Jiri Slaby wrote: On 01/11/2008

Re: [PATCH] checkpatch.pl: allow piping

2008-01-11 Thread Jiri Slaby
On 01/11/2008 12:16 PM, Stefan Richter wrote: Daniel Walker wrote: On Fri, 2008-01-11 at 10:34 +0100, Jiri Slaby wrote: On Fre, 2008-01-11 at 10:21 +0100, Jiri Slaby wrote: git show 9914cad54c79d0b89b1f066c0894f00e1344131c | ./scripts/checkpatch.pl - If somebody is hacking kernel, I think

[PATCH] checkpatch.pl: allow piping

2008-01-10 Thread Daniel Walker
A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. Signed-off-by: Daniel Walker <[EMAIL PROTECTED]> --- scripts/checkpatch.pl | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) Index:

[PATCH] checkpatch.pl: allow piping

2008-01-10 Thread Daniel Walker
A little feature addition to allow checkpatch.pl to check patches piped into it, in addition to specific file arguments. Signed-off-by: Daniel Walker [EMAIL PROTECTED] --- scripts/checkpatch.pl | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) Index: