[U-Boot] [PATCH v3 18/26] test: dm: eth: Handle failed test env cleanup

Joe Hershberger
2015-05-05T23:39:00Z

Hi Simon,

On Tue, May 5, 2015 at 3:56 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi Joe,
>
> On 3 May 2015 at 14:12, Joe Hershberger <joe.hershberger@ni.com> wrote:
>> Make sure that the env gets cleaned up after a test fails so that other
>> tests aren't affected.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger@ni.com>
>> ---
>>
>> Changes in v3:
>> -New for version 3
>>
>> Changes in v2: None
>>
>> test/dm/eth.c | 79 +++++++++++++++++++++++++++++++++++++++++------------------
>> 1 file changed, 55 insertions(+), 24 deletions(-)
>
> With this commit I get quite a few errors, like:
>
> 19: test: dm: eth: Handle failed test env cleanup
> sandbox: + sandbox
> + int retval;
> + ^
> +test/dm/built-in.o: In function `dm_test_bus_children':
> +build/../test/dm/bus.c:121: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:123: undefined reference to `ut_fail'
> +test/dm/built-in.o: In function `dm_test_bus_children_funcs':
> +build/../test/dm/bus.c:143: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:168: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:169: undefined reference to `ut_fail'

Are you sure it's this commit? This touches nothing it test/dm/bus.c
and nothing in test/ut.c (where those functions are defined). Is is
maybe a dirty build error?

Thanks,
-Joe