All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Sandeen <sandeen@sandeen.net>
To: xfs mailing list <xfs@oss.sgi.com>
Subject: [PATCH 3/3] xfstests: rename _acl_requirements to _require_acls
Date: Wed, 03 Feb 2010 15:41:01 -0600	[thread overview]
Message-ID: <4B69ED6D.6030902@sandeen.net> (raw)
In-Reply-To: <4B69EC23.8040207@sandeen.net>

Most requirement tests are named _require_foo, so just
make this more consistent.

Also remove a few redundant tests for /usr/bin/chacl
since _require_acls covers that.

Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
---

diff --git a/051 b/051
index 238abe9..12febd3 100755
--- a/051
+++ b/051
@@ -73,7 +73,6 @@ _cleanup()
 _supported_fs xfs udf
 _supported_os Linux
 
-[ -x /usr/bin/chacl ] || _notrun "chacl executable not found"
 [ -x $runas ] || _notrun "$runas executable not found"
 
 rm -f $seq.full
@@ -82,7 +81,7 @@ _setup_testdir
 
 _need_to_be_root
 _acl_setup_ids
-_acl_requirements
+_require_acls
 
 # get dir
 cd $testdir
diff --git a/053 b/053
index 439cbe6..ac2162d 100755
--- a/053
+++ b/053
@@ -42,7 +42,7 @@ _supported_fs generic
 _supported_os Linux
 
 _require_scratch
-_acl_requirements
+_require_acls
 _acl_setup_ids
 _do_die_on_error=y
 test=$SCRATCH_MNT/test
diff --git a/067 b/067
index 5ab743e..f1c211f 100755
--- a/067
+++ b/067
@@ -41,10 +41,8 @@ trap "rm -f $tmp.*; exit \$status" 0 1 2 3 15
 _supported_fs xfs
 _supported_os Linux
 
-[ -x /usr/bin/chacl ] || _notrun "chacl executable not found"
-
 _need_to_be_root
-_acl_requirements
+_require_acls
 _require_scratch
 
 # set up fs for 1K inodes
diff --git a/077 b/077
index a3d9334..ea81c07 100755
--- a/077
+++ b/077
@@ -53,7 +53,7 @@ _supported_os Linux
 [ ! -d $filler ] && _notrun "No directory to source files from"
 
 _require_scratch
-_acl_requirements
+_require_acls
 
 echo "*** create filesystem"
 
diff --git a/099 b/099
index 90af18f..b68bfcd 100755
--- a/099
+++ b/099
@@ -81,7 +81,8 @@ _supported_fs generic
 _supported_os IRIX
 
 _acl_setup_ids
-_acl_requirements
+_require_acls
+
 [ -x $runas ] || _notrun "$runas executable not found"
 
 # get dir
diff --git a/105 b/105
index 9544c66..aba1f6d 100755
--- a/105
+++ b/105
@@ -56,7 +56,7 @@ rm -f $seq.full
 _require_scratch
 
 _acl_setup_ids
-_acl_requirements
+_require_acls
 
 umount $SCRATCH_DEV >/dev/null 2>&1
 echo "*** MKFS ***"                         >>$seq.full
diff --git a/common.attr b/common.attr
index d12cc02..5019884 100644
--- a/common.attr
+++ b/common.attr
@@ -112,7 +112,7 @@ _filter_aces_notypes()
 
 # test if acl code will work
 #
-_acl_requirements()
+_require_acls()
 {
     xfsdir=$TEST_DIR
     

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  parent reply	other threads:[~2010-02-03 21:39 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-03 21:35 [PATCH 0/3] resending a few xfstests patches as a series Eric Sandeen
2010-02-03 21:37 ` [PATCH 1/3] xfstests: routine to create scratch of certain size Eric Sandeen
2010-02-05  9:37   ` Christoph Hellwig
2010-02-03 21:38 ` [PATCH 2/3] xfstests: make 053 and 077 generic Eric Sandeen
2010-02-05  9:37   ` Christoph Hellwig
2010-02-03 21:41 ` Eric Sandeen [this message]
2010-02-05  9:38   ` [PATCH 3/3] xfstests: rename _acl_requirements to _require_acls Christoph Hellwig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4B69ED6D.6030902@sandeen.net \
    --to=sandeen@sandeen.net \
    --cc=xfs@oss.sgi.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.