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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 still that sounds annoying, GIT is our default tool for handling patches and all the other tools like checkpatch.pl should honour that without any tricks. Why you don't add some detection logic to checkpatch.pl or even some new switch like --git.

regards
Antti
--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux