public inbox for fstests@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 2/2] Add some missing symlinks and hardlinks requirements.
@ 2015-08-26  9:15 Ari Sundholm
  2015-08-27  1:48 ` Dave Chinner
  0 siblings, 1 reply; 3+ messages in thread
From: Ari Sundholm @ 2015-08-26  9:15 UTC (permalink / raw)
  To: fstests

From: Ari Sundholm <ari@tuxera.com>

Signed-off-by: Ari Sundholm <ari@tuxera.com>
---
 tests/generic/067 |    1 +
 tests/generic/236 |    1 +
 tests/generic/294 |    1 +
 3 files changed, 3 insertions(+)

diff --git a/tests/generic/067 b/tests/generic/067
index 7a4840b..8322ef8 100755
--- a/tests/generic/067
+++ b/tests/generic/067
@@ -52,6 +52,7 @@ _cleanup()
 # Modify as appropriate.
 _supported_fs generic
 _supported_os Linux
+_require_test_symlinks
 _require_test
 _require_scratch
 _require_loop
diff --git a/tests/generic/236 b/tests/generic/236
index 12ea0bc..df0250f 100755
--- a/tests/generic/236
+++ b/tests/generic/236
@@ -43,6 +43,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
 _supported_fs generic
 # only Linux supports fallocate
 _supported_os Linux
+_require_test_hardlinks
 _require_test
 
 rm -f $TEST_DIR/ouch*
diff --git a/tests/generic/294 b/tests/generic/294
index 3fa6ba2..944b6a1 100755
--- a/tests/generic/294
+++ b/tests/generic/294
@@ -46,6 +46,7 @@ _cleanup()
 # Modify as appropriate.
 _supported_fs generic
 _supported_os Linux
+_require_test_hardlinks
 _require_scratch
 
 rm -f $seqres.full
-- 
1.7.10.4




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

* Re: [PATCH 2/2] Add some missing symlinks and hardlinks requirements.
  2015-08-26  9:15 [PATCH 2/2] Add some missing symlinks and hardlinks requirements Ari Sundholm
@ 2015-08-27  1:48 ` Dave Chinner
  2015-08-27 11:02   ` Ari Sundholm
  0 siblings, 1 reply; 3+ messages in thread
From: Dave Chinner @ 2015-08-27  1:48 UTC (permalink / raw)
  To: Ari Sundholm; +Cc: fstests

On Wed, Aug 26, 2015 at 12:15:48PM +0300, Ari Sundholm wrote:
> From: Ari Sundholm <ari@tuxera.com>
> 
> Signed-off-by: Ari Sundholm <ari@tuxera.com>
> ---
>  tests/generic/067 |    1 +
>  tests/generic/236 |    1 +
>  tests/generic/294 |    1 +
>  3 files changed, 3 insertions(+)
> 
> diff --git a/tests/generic/067 b/tests/generic/067
> index 7a4840b..8322ef8 100755
> --- a/tests/generic/067
> +++ b/tests/generic/067
> @@ -52,6 +52,7 @@ _cleanup()
>  # Modify as appropriate.
>  _supported_fs generic
>  _supported_os Linux
> +_require_test_symlinks
>  _require_test
>  _require_scratch
>  _require_loop
> diff --git a/tests/generic/236 b/tests/generic/236
> index 12ea0bc..df0250f 100755
> --- a/tests/generic/236
> +++ b/tests/generic/236
> @@ -43,6 +43,7 @@ trap "_cleanup; exit \$status" 0 1 2 3 15
>  _supported_fs generic
>  # only Linux supports fallocate
>  _supported_os Linux
> +_require_test_hardlinks
>  _require_test

This belongs in the previous patch that introduced the check.

Cheers,

DAve.
-- 
Dave Chinner
david@fromorbit.com

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

* Re: [PATCH 2/2] Add some missing symlinks and hardlinks requirements.
  2015-08-27  1:48 ` Dave Chinner
@ 2015-08-27 11:02   ` Ari Sundholm
  0 siblings, 0 replies; 3+ messages in thread
From: Ari Sundholm @ 2015-08-27 11:02 UTC (permalink / raw)
  To: Dave Chinner; +Cc: fstests

Hello Dave!

On Thu, 2015-08-27 at 11:48 +1000, Dave Chinner wrote:
> This belongs in the previous patch that introduced the check.

OK, I will combine the hard link part of the patches, but I think in
this case the symlink part should have its own patch as it is sort of
unrelated (even though it's a very small change by itself).

Best regards,
Ari Sundholm
ari@tuxera.com


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

end of thread, other threads:[~2015-08-27 11:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-26  9:15 [PATCH 2/2] Add some missing symlinks and hardlinks requirements Ari Sundholm
2015-08-27  1:48 ` Dave Chinner
2015-08-27 11:02   ` Ari Sundholm

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox