devtools: fix checkpatch with dash
authorGavin Hu <gavin.hu@arm.com>
Wed, 1 Aug 2018 05:22:57 +0000 (13:22 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 1 Aug 2018 14:42:53 +0000 (16:42 +0200)
When running checkpatch.sh, it generates the following error
on some linux distributions(like Debian) with Dash as the
default shell interpreter.
trap: SIGINT: bad trap

The fix is to replace SIGINT with INT signal, it works for
both bash and dash.

Fixes: 4bec48184e ("devtools: add checks for ABI symbol addition")
Cc: stable@dpdk.org
Signed-off-by: Gavin Hu <gavin.hu@arm.com>
Reviewed-by: Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
Tested-by: Vipin Varghese <vipin.varghese@intel.com>
devtools/checkpatches.sh

index 2509269..ba795ad 100755 (executable)
@@ -29,7 +29,7 @@ clean_tmp_files() {
        fi
 }
 
-trap "clean_tmp_files" SIGINT
+trap "clean_tmp_files" INT
 
 print_usage () {
        cat <<- END_OF_HELP