devtools: fix file listing in maintainers check
authorThomas Monjalon <thomas@monjalon.net>
Tue, 15 Jun 2021 12:49:49 +0000 (14:49 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 23 Jun 2021 09:18:54 +0000 (11:18 +0200)
When having multiple working trees, the main one has a .git directory
while attached trees have a .git file.
Thus the git check should work for both file and directory.

In the case there is no working tree (.git not readable), the command
"find" is used and should be able to list paths with wildcards.
Wildcards work only as shell expansion in the case of file paths,
so the quotes must be removed.

Fixes: 27c2ce563216 ("maintainers: start a Linux-style file")
Cc: stable@dpdk.org
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
devtools/check-maintainers.sh

index df3f740..71697bb 100755 (executable)
@@ -15,10 +15,10 @@ files () # <path> [<path> ...]
        if [ -z "$1" ] ; then
                return
        fi
-       if [ -d .git ] ; then
+       if [ -r .git ] ; then
                git ls-files "$1"
        else
-               find "$1" -type f |
+               find $1 -type f |
                sed 's,^\./,,'
        fi |
        # if not ended by /