Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Joe Perches
On Tue, 2011-09-06 at 19:23 +0300, Antti Palosaari wrote: > hmm, lets see. Maybe I will add --format=email as keyboard shortcut button. or add [format] pretty = email to your .gitconfig or create and use a check_commitlog shell script or... -- To unsubscribe from this list: send the li

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Antti Palosaari
On 09/06/2011 07:10 PM, Joe Perches wrote: On Tue, 2011-09-06 at 18:30 +0300, Antti Palosaari wrote: On 09/06/2011 06:15 PM, Joe Perches wrote: On Tue, 2011-09-06 at 17:41 +0300, Antti Palosaari wrote: So what is recommended way to ensure patch is correct currently? a) before commit Use check

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Joe Perches
On Tue, 2011-09-06 at 18:30 +0300, Antti Palosaari wrote: > On 09/06/2011 06:15 PM, Joe Perches wrote: > > On Tue, 2011-09-06 at 17:41 +0300, Antti Palosaari wrote: > >> So what is recommended way to ensure patch is correct currently? > >> a) before commit > > Use checkpatch. > >> b) after commit >

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Antti Palosaari
On 09/06/2011 06:15 PM, Joe Perches wrote: On Tue, 2011-09-06 at 17:41 +0300, Antti Palosaari wrote: So what is recommended way to ensure patch is correct currently? a) before commit Use checkpatch. b) after commit Make the output of the commit log look like a patch. --format=email But

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Joe Perches
On Tue, 2011-09-06 at 17:41 +0300, Antti Palosaari wrote: > So what is recommended way to ensure patch is correct currently? > a) before commit Use checkpatch. > b) after commit Make the output of the commit log look like a patch. -- To unsubscribe from this list: send the line "unsubscribe li

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Antti Palosaari
acceptable logging format for checkpatch.pl ? [crope@localhost linux]$ git show 1b19e42952963ae2a09a655f487de15b7c81c5b7 |./scripts/checkpatch.pl - WARNING: Do not use whitespace before Signed-off-by: Don't know if checkpatch used to accept that, but you can use "--format=email" to ma

Re: checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-06 Thread Bjørn Mork
g format for checkpatch.pl ? > > [crope@localhost linux]$ git show > 1b19e42952963ae2a09a655f487de15b7c81c5b7 |./scripts/checkpatch.pl - > WARNING: Do not use whitespace before Signed-off-by: Don't know if checkpatch used to accept that, but you can use "--format=email" to make it w

checkpatch.pl WARNING: Do not use whitespace before Signed-off-by:

2011-09-05 Thread Antti Palosaari
]$ git show 1b19e42952963ae2a09a655f487de15b7c81c5b7 |./scripts/checkpatch.pl - WARNING: Do not use whitespace before Signed-off-by: #10: Signed-off-by: Joe Perches WARNING: Do not use whitespace before Signed-off-by: #11: Signed-off-by: Mauro Carvalho Chehab total: 0 errors, 2 warnings