CVS commit: pkgsrc/mk/check

["Roland Illig" <rillig%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   rillig
Date:           Thu Mar 12 08:44:15 UTC 2020

Modified Files:
        pkgsrc/mk/check: check-portability.sh

Log Message:
mk/check/check-portability: check Makefile.am and Makefile.in as well

Up to now, these files had been skipped because they didn't start with a
#! line. Still, they contain shell programs, therefore it makes sense to
check them.


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.14 pkgsrc/mk/check/check-portability.sh

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/mk/check/check-portability.sh
diff -u pkgsrc/mk/check/check-portability.sh:1.13 pkgsrc/mk/check/check-portability.sh:1.14
--- pkgsrc/mk/check/check-portability.sh:1.13   Wed Mar 11 23:59:27 2020
+++ pkgsrc/mk/check/check-portability.sh        Thu Mar 12 08:44:15 2020
@@ -1,4 +1,4 @@
-# $NetBSD: check-portability.sh,v 1.13 2020/03/11 23:59:27 rillig Exp $
+# $NetBSD: check-portability.sh,v 1.14 2020/03/12 08:44:15 rillig Exp $
 #
 # This program checks all files in the current directory and any
 # subdirectories for portability issues that are likely to result in
@@ -43,12 +43,13 @@ find * -type f -print 2>/dev/null \
                eval "case \"\$fname\" in $SKIP_FILTER *.orig) skip=yes;; esac"
                [ $skip = no ] || continue
 
+               skip_shebang_test=no
                ext="${fname##*.}"
                case "$ext" in
                # A few file extensions cannot be skipped since the Makefiles
                # will be generated from these, in the configure stage, which
                # is run later.
-               (am|in)                                                 ;;
+               (am|in)                                                 skip_shebang_test=yes;;
                # echo */*/PLIST | xargs cat | sed s,'.*\.',, | sort | uniq -c | sort -nr | sed 40q
                (png|html|svg|py|h|mo|php|js|xml|rb|go|txt|3|hpp)       continue ;;
                (tfm|gif|dtd|properties|json|ogg|gz|test|result|xpm|po) continue ;;
@@ -57,6 +58,11 @@ find * -type f -print 2>/dev/null \
                (c|C|cc|cxx|f|go|pl|py|in|ac|m4)                        continue ;;
                esac
 
+               if [ $skip_shebang_test = yes ]; then
+                       check_shell "$fname"
+                       continue
+               fi
+
                case "$opsys" in
                SunOS-5.9)
                        # See also (if you can):