[PATCH 1/1] test: missing dependency for test/cmd/setexpr.c

[Simon Glass]

Hi Heinrich,

On Wed, 17 Feb 2021 at 04:58, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> test/cmd/setexpr.c cannot be linked with CONFIG_CMD_SETEXPR=n:
>
> ld.bfd: test/built-in.o: in function `setexpr_test_sub':
> test/cmd/setexpr.c:227: undefined reference to `setexpr_regex_sub'
> ld.bfd: test/built-in.o: in function `setexpr_test_backref':
> test/cmd/setexpr.c:267: undefined reference to `setexpr_regex_sub'
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>  test/cmd/Makefile | 2 +-
>  test/cmd_ut.c     | 2 ++
>  2 files changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

But I wonder what the goal is here. There is an assumption at present
that sandbox runs these tests and has the required features enabled.
What are you trying to do?


- Simon