All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP]  [PATCH] fs_racer: cleanup
@ 2016-11-23 12:00 Cui Bixuan
  2016-11-23 12:21 ` Cyril Hrubis
  0 siblings, 1 reply; 5+ messages in thread
From: Cui Bixuan @ 2016-11-23 12:00 UTC (permalink / raw)
  To: ltp

Remove the DIR at the end of test

Signed-off-by: Cui Bixuan <cuibixuan@huawei.com>
---
 testcases/kernel/fs/racer/fs_racer.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/testcases/kernel/fs/racer/fs_racer.sh b/testcases/kernel/fs/racer/fs_racer.sh
index 5c4dc69..e7d5e4b 100755
--- a/testcases/kernel/fs/racer/fs_racer.sh
+++ b/testcases/kernel/fs/racer/fs_racer.sh
@@ -86,6 +86,7 @@ call_exit()
     killall fs_racer_file_symlink.sh
     killall fs_racer_file_list.sh
     killall fs_racer_file_concat.sh
+    rm -rf $DIR >/dev/null
     exit 0
 }
 
-- 
1.8.3.4


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [LTP] [PATCH] fs_racer: cleanup
  2016-11-23 12:00 [LTP] [PATCH] fs_racer: cleanup Cui Bixuan
@ 2016-11-23 12:21 ` Cyril Hrubis
  2016-11-24  1:11   ` [LTP] [PATCH v2] " Cui Bixuan
  0 siblings, 1 reply; 5+ messages in thread
From: Cyril Hrubis @ 2016-11-23 12:21 UTC (permalink / raw)
  To: ltp

Hi!
> +    rm -rf $DIR >/dev/null

There shouldn't be any need to redirect the output to /dev/null once you
pass the -f argument to the rm. Otherwise this is obviously correct.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [LTP]  [PATCH v2] fs_racer: cleanup
  2016-11-23 12:21 ` Cyril Hrubis
@ 2016-11-24  1:11   ` Cui Bixuan
  2016-11-24  8:58     ` Cyril Hrubis
  0 siblings, 1 reply; 5+ messages in thread
From: Cui Bixuan @ 2016-11-24  1:11 UTC (permalink / raw)
  To: ltp

Remove the DIR at the end of test

Signed-off-by: Cui Bixuan <cuibixuan@huawei.com>
---
 testcases/kernel/fs/racer/fs_racer.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/testcases/kernel/fs/racer/fs_racer.sh b/testcases/kernel/fs/racer/fs_racer.sh
index 5c4dc69..aa38e86 100755
--- a/testcases/kernel/fs/racer/fs_racer.sh
+++ b/testcases/kernel/fs/racer/fs_racer.sh
@@ -86,6 +86,7 @@ call_exit()
     killall fs_racer_file_symlink.sh
     killall fs_racer_file_list.sh
     killall fs_racer_file_concat.sh
+    rm -rf $DIR
     exit 0
 }

-- 
1.8.3.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [LTP] [PATCH v2] fs_racer: cleanup
  2016-11-24  1:11   ` [LTP] [PATCH v2] " Cui Bixuan
@ 2016-11-24  8:58     ` Cyril Hrubis
  2016-11-25  3:29       ` Cui Bixuan
  0 siblings, 1 reply; 5+ messages in thread
From: Cyril Hrubis @ 2016-11-24  8:58 UTC (permalink / raw)
  To: ltp

Hi!
Pushed, thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [LTP] [PATCH v2] fs_racer: cleanup
  2016-11-24  8:58     ` Cyril Hrubis
@ 2016-11-25  3:29       ` Cui Bixuan
  0 siblings, 0 replies; 5+ messages in thread
From: Cui Bixuan @ 2016-11-25  3:29 UTC (permalink / raw)
  To: ltp

On 2016/11/24 16:58, Cyril Hrubis wrote:
> Hi!
> Pushed, thanks.
Thank you

> 


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-11-25  3:29 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-23 12:00 [LTP] [PATCH] fs_racer: cleanup Cui Bixuan
2016-11-23 12:21 ` Cyril Hrubis
2016-11-24  1:11   ` [LTP] [PATCH v2] " Cui Bixuan
2016-11-24  8:58     ` Cyril Hrubis
2016-11-25  3:29       ` Cui Bixuan

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.