[PATCH] test_env: don't strip() printenv results

Tom Rini trini at konsulko.com
Fri Jan 3 00:11:30 CET 2020


On Wed, Dec 18, 2019 at 11:37:21AM -0700, Stephen Warren wrote:

> From: Stephen Warren <swarren at nvidia.com>
> 
> get_env() was originally written to strip() the output of printenv to
> isolate the test from any whitespace changes in printenv's output.
> However, this throws away any whitespace in the variable value, which can
> cause issues when test code expects to see that whitespace. In fact,
> printenv never adds any whitespace at all, so there's no need to strip.
> 
> The strip causes a practical problem for test_env_echo_exists() if
> state_test_env.get_existent_var() happens to choose a U-Boot variable that
> contains trailing whitespace. This is true for variable boot_targets.
> 
> With Python 2, get_existent_var() never returned boot_targets so this
> issue never caused a practical problem.
> 
> With Python 3, get_existent_var does sometimes return boot_targets, no
> doubt due to Python 3's different dict hash key order implementation,
> about 0.5-2% of the time, so this test appears intermittent. With the
> strip removed, this intermittency is solved, since the test passes for all
> possible U-Boot variables.
> 
> Signed-off-by: Stephen Warren <swarren at nvidia.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200102/84f3669c/attachment.sig>


More information about the U-Boot mailing list