diff options
| author | Tom Rini <[email protected]> | 2026-05-14 15:39:22 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-05-14 15:39:22 -0600 |
| commit | 9a13d08732509fd31e9070989570e6d6bd2ad007 (patch) | |
| tree | 0489ed69b340eb610c676f1214e9218294cef0d0 /scripts/basic | |
| parent | e3e651c480c46b332f16a7555b97c6c6fd640a40 (diff) | |
| parent | 93d7dc20e80e33ace9b871de44877548642f86cb (diff) | |
Merge patch series "test: Quote variables in 'test -n' expressions across boards"
Simon Glass <[email protected]> says:
Several boards use 'test -n ${var}' (unquoted) in env scripts to detect
an empty variable and trigger a fallback or skip a hook. That works only
because of a U-Boot 'test' quirk where calls with argc < 3 returned
false; an empty variable made the expression 'test -n' (one operand) and
hit that path.
Commit 8b0619579b22 ("cmd: test: fix handling of single-argument form of
test") aligned 'test' with POSIX so those expressions flipped to true.
f7e7c55e53e8 ("cmd: test: add bug-compatibility special case for 'test
-n'") restored the old behaviour for the exact 'test -n' (one arg) case,
so the boards are not broken at runtime today, but the reliance on a
bug-compat path isn't great.
This series updates various boards to quote each affected variable so
the emptiness check is explicit and does not depend on the
special-case path.
Each commit carries a Fixes: tag pointing at 8b0619579b22
Link: https://lore.kernel.org/r/[email protected]
Diffstat (limited to 'scripts/basic')
0 files changed, 0 insertions, 0 deletions
