diff mbox series

[1/2] isar-sstate: lint task flags as well

Message ID 20260924064547.4017118-1-felix.moessbauer@siemens.com
State Under Review
Headers show
Series [1/2] isar-sstate: lint task flags as well | expand

Commit Message

MOESSBAUER, Felix Sept. 24, 2026, 6:45 a.m. UTC 
We previously only linted all uppercase variables, but by that we missed
absolute path in task flags (e.g. do_unpack[cleandirs]). Lint them as
well.

The exact reason why we only linted the uppercase variables remains
unclear. On todays caches, this neither has a significant performance
impact, nor does it report false positives.

Signed-off-by: Felix Moessbauer <felix.moessbauer@siemens.com>
---
 scripts/isar-sstate | 2 --
 1 file changed, 2 deletions(-)
diff mbox series

Patch

diff --git a/scripts/isar-sstate b/scripts/isar-sstate
index ada154e2..5d7b8ebc 100755
--- a/scripts/isar-sstate
+++ b/scripts/isar-sstate
@@ -894,8 +894,6 @@  def sstate_lint(target, verbose, sources_dir, build_dir, exit_code, pedantic, li
 
         pn_issues = []
         for name, val in sigdata['varvals'].items():
-            if not name[0].isupper():
-                continue
             if sigdata['basehash_ignore_vars'] and name in sigdata['basehash_ignore_vars'] or \
                sigdata['taskhash_ignore_tasks'] and name in sigdata['taskhash_ignore_tasks'] or \
                name in ADDITIONAL_IGNORED_VARNAMES: