Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030303Ab2KXJpr (ORCPT ); Sat, 24 Nov 2012 04:45:47 -0500 Received: from mail-ee0-f46.google.com ([74.125.83.46]:64976 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1030191Ab2KXJpo (ORCPT ); Sat, 24 Nov 2012 04:45:44 -0500 MIME-Version: 1.0 In-Reply-To: <20121124083140.241502487@redhat.com> References: <20121124082918.671430919@redhat.com> <20121124083140.241502487@redhat.com> Date: Sat, 24 Nov 2012 11:45:43 +0200 X-Google-Sender-Auth: N1i1Ygio1xauBexXMaF5z_zIc5M Message-ID: Subject: Re: [patch 7/8] kcmp selftests: print fail status instead of cause make error From: Pekka Enberg To: dyoung@redhat.com Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1252 Lines: 34 On Sat, Nov 24, 2012 at 10:29 AM, wrote: > In case kcmp_test exit non zero value it will cause make error. > Better way is just print the test failure status. > > Signed-off-by: Dave Young Reviewed-by: Pekka Enberg > --- > tools/testing/selftests/kcmp/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- linux-2.6.orig/tools/testing/selftests/kcmp/Makefile 2012-11-23 23:10:18.000000000 +0800 > +++ linux-2.6/tools/testing/selftests/kcmp/Makefile 2012-11-23 23:11:52.474414896 +0800 > @@ -22,7 +22,7 @@ else > endif > > run_tests: all > - ./kcmp_test > + @./kcmp_test || echo "kcmp_test: [FAIL]" > > clean: > rm -fr ./run_test > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/