diff mbox

[2/2] Add some missing symlinks and hardlinks requirements.

Message ID 1440580548.29614.6.camel@ari-macbook (mailing list archive)
State New, archived
Headers show

Commit Message

Ari Sundholm Aug. 26, 2015, 9:15 a.m. UTC
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(+)

Comments

Dave Chinner Aug. 27, 2015, 1:48 a.m. UTC | #1
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.
Ari Sundholm Aug. 27, 2015, 11:02 a.m. UTC | #2
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

--
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
diff mbox

Patch

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