devtools: fix directory filter in forbidden token check
authorDavid Marchand <david.marchand@redhat.com>
Tue, 10 Nov 2020 15:40:05 +0000 (16:40 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Fri, 13 Nov 2020 08:35:04 +0000 (09:35 +0100)
checkpatches.sh current complains on a patch [1] adding
ALLOW_EXPERIMENTAL_API in an example while this check is for app, lib
and drivers directories:

Warning in examples/ethtool/ethtool-app/Makefile:
Using experimental build flag for in-tree compilation

The regexp on entering files concerned by this filter is incorrect.
In the [1] case, the file full name is matched against "app" rather than
"+++ b/app".

1: https://patchwork.dpdk.org/patch/83902/

Fixes: 7413e7f2aeb3 ("devtools: alert on new calls to exit from libs")
Cc: stable@dpdk.org
Signed-off-by: David Marchand <david.marchand@redhat.com>
devtools/check-forbidden-tokens.awk

index f86cbe8..61ba707 100755 (executable)
@@ -54,7 +54,7 @@ BEGIN {
        }
        for (i in deny_folders) {
                re = "^\\+\\+\\+ b/" deny_folders[i];
-               if ($0 ~ deny_folders[i]) {
+               if ($0 ~ re) {
                        in_file = 1
                        last_file = $0
                }