Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-21 Thread Prarit Bhargava
On 10/20/2014 10:11 PM, Joe Perches wrote: > On Mon, 2014-10-20 at 20:46 -0400, Prarit Bhargava wrote: >> >> On 10/20/2014 07:11 PM, Joe Perches wrote: >>> On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: I tested this using both lower and upper case 'c' with the following commit

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-21 Thread Prarit Bhargava
On 10/20/2014 10:11 PM, Joe Perches wrote: On Mon, 2014-10-20 at 20:46 -0400, Prarit Bhargava wrote: On 10/20/2014 07:11 PM, Joe Perches wrote: On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: I tested this using both lower and upper case 'c' with the following commit text: []

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Joe Perches
On Mon, 2014-10-20 at 20:46 -0400, Prarit Bhargava wrote: > > On 10/20/2014 07:11 PM, Joe Perches wrote: > > On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: > >> I tested this using both lower and upper case 'c' with the following commit > >> text: > > [] > > > > I think the patch

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Prarit Bhargava
On 10/20/2014 07:11 PM, Joe Perches wrote: > On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: >> I tested this using both lower and upper case 'c' with the following commit >> text: > [] > > I think the patch subject be something like: > > "[PATCH] checkpatch: improve commit id/desc

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Joe Perches
On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: > I tested this using both lower and upper case 'c' with the following commit > text: [] I think the patch subject be something like: "[PATCH] checkpatch: improve commit id/desc style checking in commit message" The [v2] goes in the

[PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Prarit Bhargava
I tested this using both lower and upper case 'c' with the following commit text: Derp, derpy derp. Derps derpy derpy derp derp derp derp. Some other text besides derp. Stuff. 11 chars, SHOULD FAIL commit 1234567890a 12 chars commit 1234567890ab 13 chars commit 1234567890abc 14 chars commit

[PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Prarit Bhargava
I tested this using both lower and upper case 'c' with the following commit text: Derp, derpy derp. Derps derpy derpy derp derp derp derp. Some other text besides derp. Stuff. 11 chars, SHOULD FAIL commit 1234567890a 12 chars commit 1234567890ab 13 chars commit 1234567890abc 14 chars commit

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Joe Perches
On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: I tested this using both lower and upper case 'c' with the following commit text: [] I think the patch subject be something like: [PATCH] checkpatch: improve commit id/desc style checking in commit message The [v2] goes in the subject

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Prarit Bhargava
On 10/20/2014 07:11 PM, Joe Perches wrote: On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: I tested this using both lower and upper case 'c' with the following commit text: [] I think the patch subject be something like: [PATCH] checkpatch: improve commit id/desc style

Re: [PATCH] scripts, checkpatch.pl, provide a better output message for commit id format [v2]

2014-10-20 Thread Joe Perches
On Mon, 2014-10-20 at 20:46 -0400, Prarit Bhargava wrote: On 10/20/2014 07:11 PM, Joe Perches wrote: On Mon, 2014-10-20 at 18:49 -0400, Prarit Bhargava wrote: I tested this using both lower and upper case 'c' with the following commit text: [] I think the patch subject be something