Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Mathieu Desnoyers
- On Oct 3, 2015, at 1:55 PM, Josh Triplett j...@joshtriplett.org wrote: > On Sat, Oct 03, 2015 at 02:11:57PM +, Mathieu Desnoyers wrote: >> - On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: >> >> > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: >>

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Josh Triplett
On Sat, Oct 03, 2015 at 02:11:57PM +, Mathieu Desnoyers wrote: > - On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: > > > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: > >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com > >>

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Mathieu Desnoyers
- On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com wrote: >> >> > Some test's Makefile using "$(RM)" while the other's >> > using "rm

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Mathieu Desnoyers
- On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com wrote: >> >> > Some test's Makefile using "$(RM)" while the other's >> > using "rm

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Josh Triplett
On Sat, Oct 03, 2015 at 02:11:57PM +, Mathieu Desnoyers wrote: > - On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: > > > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: > >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com > >>

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-03 Thread Mathieu Desnoyers
- On Oct 3, 2015, at 1:55 PM, Josh Triplett j...@joshtriplett.org wrote: > On Sat, Oct 03, 2015 at 02:11:57PM +, Mathieu Desnoyers wrote: >> - On Oct 3, 2015, at 12:38 AM, dvhart dvh...@infradead.org wrote: >> >> > On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: >>

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-02 Thread Darren Hart
On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: > - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. > > I agree that this

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-10-02 Thread Darren Hart
On Mon, Sep 28, 2015 at 03:16:53AM +, Mathieu Desnoyers wrote: > - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. > > I agree that this

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-29 Thread Kees Cook
On Tue, Sep 29, 2015 at 2:57 AM, Michael Ellerman wrote: > On Mon, 2015-09-28 at 03:16 +, Mathieu Desnoyers wrote: >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com wrote: >> >> > Some test's Makefile using "$(RM)" while the other's >> > using "rm -f". It is better to

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-29 Thread Michael Ellerman
On Mon, 2015-09-28 at 03:16 +, Mathieu Desnoyers wrote: > - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. > > I agree that this

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-29 Thread Michael Ellerman
On Mon, 2015-09-28 at 03:16 +, Mathieu Desnoyers wrote: > - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. > > I agree that this

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-29 Thread Kees Cook
On Tue, Sep 29, 2015 at 2:57 AM, Michael Ellerman wrote: > On Mon, 2015-09-28 at 03:16 +, Mathieu Desnoyers wrote: >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com wrote: >> >> > Some test's Makefile using "$(RM)" while the other's >> > using "rm

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-28 Thread Yuan Sun
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 ---

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-28 Thread Yuan Sun
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

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-27 Thread Cam Hutchison
Mathieu Desnoyers writes: >- On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. >I agree that this disparity appears to be unwanted. We >should

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-27 Thread Mathieu Desnoyers
- On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. I agree that this disparity appears to be unwanted. We should settle on one or the other. > >

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-27 Thread Mathieu Desnoyers
- On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. I agree that this disparity appears to be unwanted. We should settle on one or the other. > >

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-27 Thread Cam Hutchison
Mathieu Desnoyers writes: >- On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com 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. >I agree that this disparity