From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756813AbbI1HdJ (ORCPT ); Mon, 28 Sep 2015 03:33:09 -0400 Received: from szxga03-in.huawei.com ([119.145.14.66]:4150 "EHLO szxga03-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756412AbbI1HdG (ORCPT ); Mon, 28 Sep 2015 03:33:06 -0400 Message-ID: <5608EBBE.6090800@huawei.com> Date: Mon, 28 Sep 2015 15:26:54 +0800 From: Yuan Sun User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:31.0) Gecko/20100101 Thunderbird/31.1.2 MIME-Version: 1.0 To: Wang Long , , , CC: , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH] kselftest: replace $(RM) with rm -f command References: <20150925154415.GC38748@vmdeb7> <1443406217-137773-1-git-send-email-long.wanglong@huawei.com> In-Reply-To: <1443406217-137773-1-git-send-email-long.wanglong@huawei.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.111.50.244] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090204.5608EBCE.000D,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-05-26 15:14:31, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: e38b35be5f4f0240f19e46e20d038ada Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org See the in-line comment. On 2015/9/28 10:10, Wang Long wrote: > Some test's Makefile using "$(RM)" while the other's > using "rm -f". It is better to use one of them in all > tests. > > "rm -f" is better, because it is less magic, and everyone > konws what is does. > > Signed-off-by: Wang Long > --- > tools/testing/selftests/capabilities/Makefile | 2 +- > tools/testing/selftests/kcmp/Makefile | 2 +- > tools/testing/selftests/membarrier/Makefile | 2 +- > tools/testing/selftests/memfd/Makefile | 2 +- > tools/testing/selftests/net/Makefile | 2 +- > tools/testing/selftests/seccomp/Makefile | 2 +- > tools/testing/selftests/size/Makefile | 2 +- > tools/testing/selftests/vm/Makefile | 2 +- > tools/testing/selftests/x86/Makefile | 2 +- > tools/testing/selftests/zram/Makefile | 2 +- > 10 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/tools/testing/selftests/capabilities/Makefile b/tools/testing/selftests/capabilities/Makefile > index 8c8f0c1..dcc1972 100644 > --- a/tools/testing/selftests/capabilities/Makefile > +++ b/tools/testing/selftests/capabilities/Makefile > @@ -12,7 +12,7 @@ CFLAGS := -O2 -g -std=gnu99 -Wall -lcap-ng > all: $(TARGETS) > > clean: > - $(RM) $(TARGETS) > + rm -f $(TARGETS) > > $(TARGETS): %: %.c > $(CC) -o $@ $(CFLAGS) $(EXTRA_CFLAGS) $^ -lrt -ldl > diff --git a/tools/testing/selftests/kcmp/Makefile b/tools/testing/selftests/kcmp/Makefile > index 2ae7450..2deaee0 100644 > --- a/tools/testing/selftests/kcmp/Makefile > +++ b/tools/testing/selftests/kcmp/Makefile > @@ -7,4 +7,4 @@ TEST_PROGS := kcmp_test > include ../lib.mk > > clean: > - $(RM) kcmp_test kcmp-test-file > + rn -f kcmp_test kcmp-test-file It should be rm, not rn. > diff --git a/tools/testing/selftests/membarrier/Makefile b/tools/testing/selftests/membarrier/Makefile > index a1a9708..f23fc58 100644 > --- a/tools/testing/selftests/membarrier/Makefile > +++ b/tools/testing/selftests/membarrier/Makefile > @@ -7,4 +7,4 @@ all: $(TEST_PROGS) > include ../lib.mk > > clean: > - $(RM) $(TEST_PROGS) > + rm -f $(TEST_PROGS) > diff --git a/tools/testing/selftests/memfd/Makefile b/tools/testing/selftests/memfd/Makefile > index 3e7eb79..068fa93 100644 > --- a/tools/testing/selftests/memfd/Makefile > +++ b/tools/testing/selftests/memfd/Makefile > @@ -19,4 +19,4 @@ run_fuse: build_fuse > @./run_fuse_test.sh || echo "fuse_test: [FAIL]" > > clean: > - $(RM) memfd_test fuse_test > + rm -f memfd_test fuse_test > diff --git a/tools/testing/selftests/net/Makefile b/tools/testing/selftests/net/Makefile > index fac4782..ec7eaa4 100644 > --- a/tools/testing/selftests/net/Makefile > +++ b/tools/testing/selftests/net/Makefile > @@ -16,4 +16,4 @@ TEST_FILES := $(NET_PROGS) > include ../lib.mk > > clean: > - $(RM) $(NET_PROGS) > + rm -f $(NET_PROGS) > diff --git a/tools/testing/selftests/seccomp/Makefile b/tools/testing/selftests/seccomp/Makefile > index 8401e87..c16072a 100644 > --- a/tools/testing/selftests/seccomp/Makefile > +++ b/tools/testing/selftests/seccomp/Makefile > @@ -7,4 +7,4 @@ all: $(TEST_PROGS) > include ../lib.mk > > clean: > - $(RM) $(TEST_PROGS) > + rm -f $(TEST_PROGS) > diff --git a/tools/testing/selftests/size/Makefile b/tools/testing/selftests/size/Makefile > index bbd0b53..cefe914 100644 > --- a/tools/testing/selftests/size/Makefile > +++ b/tools/testing/selftests/size/Makefile > @@ -8,4 +8,4 @@ TEST_PROGS := get_size > include ../lib.mk > > clean: > - $(RM) get_size > + rm -f get_size > diff --git a/tools/testing/selftests/vm/Makefile b/tools/testing/selftests/vm/Makefile > index 3c53cac..26663c7 100644 > --- a/tools/testing/selftests/vm/Makefile > +++ b/tools/testing/selftests/vm/Makefile > @@ -24,4 +24,4 @@ TEST_FILES := $(BINARIES) > include ../lib.mk > > clean: > - $(RM) $(BINARIES) > + rm -f $(BINARIES) > diff --git a/tools/testing/selftests/x86/Makefile b/tools/testing/selftests/x86/Makefile > index 29089b2..48b2406 100644 > --- a/tools/testing/selftests/x86/Makefile > +++ b/tools/testing/selftests/x86/Makefile > @@ -32,7 +32,7 @@ all_32: $(BINARIES_32) > all_64: $(BINARIES_64) > > clean: > - $(RM) $(BINARIES_32) $(BINARIES_64) > + rm -f $(BINARIES_32) $(BINARIES_64) > > $(TARGETS_C_32BIT_ALL:%=%_32): %_32: %.c > $(CC) -m32 -o $@ $(CFLAGS) $(EXTRA_CFLAGS) $^ -lrt -ldl > diff --git a/tools/testing/selftests/zram/Makefile b/tools/testing/selftests/zram/Makefile > index 29d8034..e1591c8 100644 > --- a/tools/testing/selftests/zram/Makefile > +++ b/tools/testing/selftests/zram/Makefile > @@ -6,4 +6,4 @@ TEST_FILES := zram01.sh zram02.sh zram_lib.sh > include ../lib.mk > > clean: > - $(RM) err.log > + rm -f err.log