devtools: make checkpatch cleaner for renamed files
authorDavid Hunt <david.hunt@intel.com>
Mon, 20 Feb 2017 04:08:33 +0000 (04:08 +0000)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Tue, 21 Feb 2017 11:24:11 +0000 (12:24 +0100)
When a file is renamed, a normal diff will include all the code of
the renamed file, and checkpatch will find warnings and errors,
even though it's just a rename.

This change will result in a 'rename' line in the diff, resulting
in a much cleaner checkpatches result.

Signed-off-by: David Hunt <david.hunt@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
devtools/checkpatches.sh

index cfe262b..78e155e 100755 (executable)
@@ -89,7 +89,7 @@ check () { # <patch> <commit> <title>
        if [ -n "$1" ] ; then
                report=$($DPDK_CHECKPATCH_PATH $options "$1" 2>/dev/null)
        elif [ -n "$2" ] ; then
-               report=$(git format-patch --no-stat --stdout -1 $commit |
+               report=$(git format-patch --find-renames --no-stat --stdout -1 $commit |
                        $DPDK_CHECKPATCH_PATH $options - 2>/dev/null)
        else
                report=$($DPDK_CHECKPATCH_PATH $options - 2>/dev/null)