2012-11-24 08:31:34

by Dave Young

[permalink] [raw]
Subject: [patch 8/8] breakpoint selftests: print failure status instead of cause make error

In case breakpoint test exit non zero value it will cause make error.
Better way is just print the test failure status.

Signed-off-by: Dave Young <[email protected]>
---
tools/testing/selftests/breakpoints/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-2.6.orig/tools/testing/selftests/breakpoints/Makefile 2012-11-23 22:37:04.749058409 +0800
+++ linux-2.6/tools/testing/selftests/breakpoints/Makefile 2012-11-23 22:38:43.221858269 +0800
@@ -17,7 +17,7 @@ else
endif

run_tests:
- ./breakpoint_test
+ @./breakpoint_test || echo "breakpoints selftests: [FAIL]"

clean:
rm -fr breakpoint_test


2012-11-24 09:46:01

by Pekka Enberg

[permalink] [raw]
Subject: Re: [patch 8/8] breakpoint selftests: print failure status instead of cause make error

On Sat, Nov 24, 2012 at 10:29 AM, <[email protected]> wrote:
> In case breakpoint test exit non zero value it will cause make error.
> Better way is just print the test failure status.
>
> Signed-off-by: Dave Young <[email protected]>

Reviewed-by: Pekka Enberg <[email protected]>

> ---
> tools/testing/selftests/breakpoints/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-2.6.orig/tools/testing/selftests/breakpoints/Makefile 2012-11-23 22:37:04.749058409 +0800
> +++ linux-2.6/tools/testing/selftests/breakpoints/Makefile 2012-11-23 22:38:43.221858269 +0800
> @@ -17,7 +17,7 @@ else
> endif
>
> run_tests:
> - ./breakpoint_test
> + @./breakpoint_test || echo "breakpoints selftests: [FAIL]"
>
> clean:
> rm -fr breakpoint_test
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/