[ptxdist] [PATCH 2/2] ptxd_make_world_lint: allow single linters to be run

[Roland Hieber <rhi@pengutronix.de>]

Some linters can take a while to run. Speed up the run-edit loop during
recipe development by making it possible to run a single linter, and add
a way of listing available linters.

Signed-off-by: Roland Hieber <rhi@pengutronix.de>
---
 bin/ptxdist                         |  5 ++++-
 doc/ref_parameter.rst               |  4 +++-
 scripts/lib/ptxd_make_world_lint.sh | 30 ++++++++++++++++++++++++-----
 3 files changed, 32 insertions(+), 7 deletions(-)

diff --git a/bin/ptxdist b/bin/ptxdist
index c41065311f0c..97370d92fc03 100755
--- a/bin/ptxdist
+++ b/bin/ptxdist
@@ -776,7 +776,8 @@ Misc:
   printnext <var>		assumes that the contents of <var> is another
 				variable and print the contents of this variable
   licensecheck			check md5sums of license files for all packages
-  lint				run some basic checks for the bsp and PTXdist
+  lint [<linter>]		run some basic checks for the BSP and PTXdist
+				(see 'lint list' for a list of available linters)
   list-packages			print a list of all selected packages
   local-src <pkg> [<directory>]	overwrite a package source with a locally provided
 				directory containing the sourcecode.
@@ -1831,6 +1832,8 @@ EOF
 			;;
 		lint)
 			export PTXDIST_GEN_ALL=1
+			export PTXDIST_LINT_TARGET=${1:-all}
+			shift
 			check_premake_compiler &&
 			ptxd_make_log ptxdist-lint &&
 			if [ -e "${PTXDIST_TEMPDIR}/lint-failed" ]; then
diff --git a/doc/ref_parameter.rst b/doc/ref_parameter.rst
index a5b9bfe9eca6..3b49ee68b7ce 100644
--- a/doc/ref_parameter.rst
+++ b/doc/ref_parameter.rst
@@ -250,10 +250,12 @@ Misc Actions
   For the specified package (or all selected packages), check the MD5 sums
   of license files.
 
-``lint``
+``lint [<linter>]``
   check the BSP and PTXdist for all kinds of issues. These are not checks
   for things that cause build errors. Instead the checks look for
   inconsistencies that may cause hidden problems.
+  ``<linter>`` can be a single linter to run, or ``list`` to print a list of
+  available linters, or ``all`` to run all linters (the default).
 
 ``list-packages``
   print a list of all selected packages. This list does not include the
diff --git a/scripts/lib/ptxd_make_world_lint.sh b/scripts/lib/ptxd_make_world_lint.sh
index 06926e58d614..a08198411292 100644
--- a/scripts/lib/ptxd_make_world_lint.sh
+++ b/scripts/lib/ptxd_make_world_lint.sh
@@ -199,10 +199,30 @@ export -f ptxd_make_world_lint_credits
 PTXDIST_LINT_COMMANDS="${PTXDIST_LINT_COMMANDS} credits"
 
 ptxd_make_world_lint() {
-    local command
-
-    for command in ${PTXDIST_LINT_COMMANDS}; do
-	ptxd_make_world_lint_${command}
-    done
+    local command done
+
+    case "${PTXDIST_LINT_TARGET}" in
+        list|help)
+            echo "Available linters: ${PTXDIST_LINT_COMMANDS}"
+            return
+            ;;
+        all)
+            for command in ${PTXDIST_LINT_COMMANDS}; do
+                ptxd_make_world_lint_${command}
+            done
+            ;;
+        *)
+            for command in ${PTXDIST_LINT_COMMANDS}; do
+                if [ "${PTXDIST_LINT_TARGET}" = "${command}" ]; then
+                    done=1
+                    ptxd_make_world_lint_${command}
+                fi
+            done
+            if [ -z "${done}" ]; then
+                ptxd_bailout "No such linter: '${PTXDIST_LINT_TARGET}'" \
+                    "Available linters: ${PTXDIST_LINT_COMMANDS}"
+            fi
+            ;;
+    esac
 }
 export -f ptxd_make_world_lint
-- 
2.29.2


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de