RE: [PATCH] checkpatch: Add support to check already applied git commits
Hi, > From: Joe Perches [mailto:j...@perches.com] > Sent: Monday, April 25, 2016 7:12 AM > To: Andrew Morton; Andy Whitcroft > > Cc: Du, Changbin ; linux-kernel@vger.kernel.org > Subject: [PATCH] checkpatch: Add support to check already applied git > commits > > It's sometimes useful to scan already committed patches. > > Add --git to scan specific or multiple commits. > > Single commits are scanned with > --git > Multiple commits are scanned with > --git > --git - > > Signed-off-by: "Du, Changbin" > Signed-off-by: Joe Perches > --- > > A few miscellaneous improvements to Changbin's original patch: > > o Don't exec git for each -, > use a single "git log - " > o Consolidate the git exec for the and - variants > o Output 12 character commit hash ids > o Don't scan git commit merges > o Use -M to reduce the size of rename commits > Thanks, it has been much more better now. I like this new one. This is my first time write a Perl script. :) Also thanks for Sebastian's explanation, I got this idea just because I had the same use case with you. Best Regards, Du, Changbin
RE: [PATCH] checkpatch: Add support to check already applied git commits
Hi, > From: Joe Perches [mailto:j...@perches.com] > Sent: Monday, April 25, 2016 7:12 AM > To: Andrew Morton ; Andy Whitcroft > > Cc: Du, Changbin ; linux-kernel@vger.kernel.org > Subject: [PATCH] checkpatch: Add support to check already applied git > commits > > It's sometimes useful to scan already committed patches. > > Add --git to scan specific or multiple commits. > > Single commits are scanned with > --git > Multiple commits are scanned with > --git > --git - > > Signed-off-by: "Du, Changbin" > Signed-off-by: Joe Perches > --- > > A few miscellaneous improvements to Changbin's original patch: > > o Don't exec git for each -, > use a single "git log - " > o Consolidate the git exec for the and - variants > o Output 12 character commit hash ids > o Don't scan git commit merges > o Use -M to reduce the size of rename commits > Thanks, it has been much more better now. I like this new one. This is my first time write a Perl script. :) Also thanks for Sebastian's explanation, I got this idea just because I had the same use case with you. Best Regards, Du, Changbin