On 3 October 2015 at 20:56, Stephen Warren <[email protected]> wrote:
> In the following snippet:
>
> if [ ! -x `which $prereq` ]; then
>
> When $prereq does not exist, `which $prereq` evaluates to the empty string,
> which results in *no* argument being passed to the -x operator, which then
> evaluates to true, which is the equivalent of the prereq having been found. In
> order for this to fail as expected, we must pass an empty argument, which then
> causes -x to fail. Do this by wrapping the `` in quotes so there's always an
> argument to -x, even if the value of the argument is zero-length.
>
> Signed-off-by: Stephen Warren <[email protected]>
> ---
> v2: New patch.
> ---
>  test/fs/fs-test.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Simon Glass <[email protected]>
Tested-by: Simon Glass <[email protected]>
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to