All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] busybox compat: use stat -c instead of stat --format
@ 2015-08-25 10:06 Ari Sundholm
  2015-08-26  6:32 ` Eryu Guan
  0 siblings, 1 reply; 2+ messages in thread
From: Ari Sundholm @ 2015-08-25 10:06 UTC (permalink / raw)
  To: fstests

From: Ari Sundholm <ari@tuxera.com>

Signed-off-by: Ari Sundholm <ari@tuxera.com>
---
 tests/generic/040 |    4 ++--
 tests/generic/041 |    2 +-
 tests/generic/104 |    4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/generic/040 b/tests/generic/040
index c841fbc..b44a433 100755
--- a/tests/generic/040
+++ b/tests/generic/040
@@ -122,9 +122,9 @@ _mount_flakey
 # The fstests framework automatically calls fsck after a test is run, so we
 # don't need to call fsck explicitly here.
 
-echo "Link count before rm foo_link_*: $(stat --format=%h $SCRATCH_MNT/foo)"
+echo "Link count before rm foo_link_*: $(stat -c %h $SCRATCH_MNT/foo)"
 rm -f $SCRATCH_MNT/foo_link_*
-echo "Link count after rm foo_link_*: $(stat --format=%h $SCRATCH_MNT/foo)"
+echo "Link count after rm foo_link_*: $(stat -c %h $SCRATCH_MNT/foo)"
 cat $SCRATCH_MNT/foo
 
 status=0
diff --git a/tests/generic/041 b/tests/generic/041
index f38b662..50fb26f 100755
--- a/tests/generic/041
+++ b/tests/generic/041
@@ -116,7 +116,7 @@ _mount_flakey
 # the hard links and read the file's data. This is just to verify we don't
 # get stale file handle errors (due to dangling directory index entries that
 # point to inodes that no longer exist).
-echo "Link count: $(stat --format=%h $SCRATCH_MNT/foo)"
+echo "Link count: $(stat -c %h $SCRATCH_MNT/foo)"
 [ -f $SCRATCH_MNT/foo ] || echo "Link foo is missing"
 for ((i = 1; i <= 3003; i++)); do
 	name=foo_link_`printf "%04d" $i`
diff --git a/tests/generic/104 b/tests/generic/104
index eeb7363..fef3583 100755
--- a/tests/generic/104
+++ b/tests/generic/104
@@ -81,8 +81,8 @@ _load_flakey_table $FLAKEY_ALLOW_WRITES
 _mount_flakey
 
 # Now verify both our files have a link count of 2.
-echo "Link count for file foo: $(stat --format=%h $SCRATCH_MNT/testdir/foo)"
-echo "Link count for file bar: $(stat --format=%h $SCRATCH_MNT/testdir/bar)"
+echo "Link count for file foo: $(stat -c %h $SCRATCH_MNT/testdir/foo)"
+echo "Link count for file bar: $(stat -c %h $SCRATCH_MNT/testdir/bar)"
 
 # We should be able to remove all the links of our files in testdir, and after
 # that the parent directory should become empty and therefore possible to
-- 
1.7.10.4



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

* Re: [PATCH] busybox compat: use stat -c instead of stat --format
  2015-08-25 10:06 [PATCH] busybox compat: use stat -c instead of stat --format Ari Sundholm
@ 2015-08-26  6:32 ` Eryu Guan
  0 siblings, 0 replies; 2+ messages in thread
From: Eryu Guan @ 2015-08-26  6:32 UTC (permalink / raw)
  To: Ari Sundholm; +Cc: fstests

On Tue, Aug 25, 2015 at 01:06:43PM +0300, Ari Sundholm wrote:
> From: Ari Sundholm <ari@tuxera.com>
> 
> Signed-off-by: Ari Sundholm <ari@tuxera.com>

Looks good to me.

Reviewed-by: Eryu Guan <eguan@redhat.com>

> ---
>  tests/generic/040 |    4 ++--
>  tests/generic/041 |    2 +-
>  tests/generic/104 |    4 ++--
>  3 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/tests/generic/040 b/tests/generic/040
> index c841fbc..b44a433 100755
> --- a/tests/generic/040
> +++ b/tests/generic/040
> @@ -122,9 +122,9 @@ _mount_flakey
>  # The fstests framework automatically calls fsck after a test is run, so we
>  # don't need to call fsck explicitly here.
>  
> -echo "Link count before rm foo_link_*: $(stat --format=%h $SCRATCH_MNT/foo)"
> +echo "Link count before rm foo_link_*: $(stat -c %h $SCRATCH_MNT/foo)"
>  rm -f $SCRATCH_MNT/foo_link_*
> -echo "Link count after rm foo_link_*: $(stat --format=%h $SCRATCH_MNT/foo)"
> +echo "Link count after rm foo_link_*: $(stat -c %h $SCRATCH_MNT/foo)"
>  cat $SCRATCH_MNT/foo
>  
>  status=0
> diff --git a/tests/generic/041 b/tests/generic/041
> index f38b662..50fb26f 100755
> --- a/tests/generic/041
> +++ b/tests/generic/041
> @@ -116,7 +116,7 @@ _mount_flakey
>  # the hard links and read the file's data. This is just to verify we don't
>  # get stale file handle errors (due to dangling directory index entries that
>  # point to inodes that no longer exist).
> -echo "Link count: $(stat --format=%h $SCRATCH_MNT/foo)"
> +echo "Link count: $(stat -c %h $SCRATCH_MNT/foo)"
>  [ -f $SCRATCH_MNT/foo ] || echo "Link foo is missing"
>  for ((i = 1; i <= 3003; i++)); do
>  	name=foo_link_`printf "%04d" $i`
> diff --git a/tests/generic/104 b/tests/generic/104
> index eeb7363..fef3583 100755
> --- a/tests/generic/104
> +++ b/tests/generic/104
> @@ -81,8 +81,8 @@ _load_flakey_table $FLAKEY_ALLOW_WRITES
>  _mount_flakey
>  
>  # Now verify both our files have a link count of 2.
> -echo "Link count for file foo: $(stat --format=%h $SCRATCH_MNT/testdir/foo)"
> -echo "Link count for file bar: $(stat --format=%h $SCRATCH_MNT/testdir/bar)"
> +echo "Link count for file foo: $(stat -c %h $SCRATCH_MNT/testdir/foo)"
> +echo "Link count for file bar: $(stat -c %h $SCRATCH_MNT/testdir/bar)"
>  
>  # We should be able to remove all the links of our files in testdir, and after
>  # that the parent directory should become empty and therefore possible to
> -- 
> 1.7.10.4
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

end of thread, other threads:[~2015-08-26  6:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-08-25 10:06 [PATCH] busybox compat: use stat -c instead of stat --format Ari Sundholm
2015-08-26  6:32 ` Eryu Guan

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.