summaryrefslogtreecommitdiff
path: root/tests/install
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2013-04-12 22:57:45 +0100
committerPádraig Brady <P@draigBrady.com>2013-04-14 01:03:56 +0100
commit1dd8a33169b69716c4a4b92dbe1bd99336d92b23 (patch)
treedee302fc0b33752491e18705c57eb830d8f7e3a3 /tests/install
parentc8ac385299950ba84eb8c33f7e32e4d85b18e3ff (diff)
downloadcoreutils-1dd8a33169b69716c4a4b92dbe1bd99336d92b23.tar.xz
tests: avoid false failures with non default groups
On OS X it was seen that the group ID used for new files, are set to a that of the directory rather than the current user. It's not currently understood when this happens, but it was confirmed that ACLs, extended attributes and setgid bits are _not_ involved. * init.cfg (skip_if_nondefault_group_): A new function to detect and avoid this situation. Document with links to the discussions for hopefully future clarification. * tests/install/install-C-root.sh: Use the new function. * tests/install/install-C-selinux.sh: Likewise. * tests/install/install-C.sh: Likewise.
Diffstat (limited to 'tests/install')
-rwxr-xr-xtests/install/install-C-root.sh1
-rwxr-xr-xtests/install/install-C-selinux.sh2
-rwxr-xr-xtests/install/install-C.sh25
3 files changed, 27 insertions, 1 deletions
diff --git a/tests/install/install-C-root.sh b/tests/install/install-C-root.sh
index d8a8d846c..6e6d7eb9d 100755
--- a/tests/install/install-C-root.sh
+++ b/tests/install/install-C-root.sh
@@ -20,6 +20,7 @@
print_ver_ ginstall
require_root_
skip_if_setgid_
+skip_if_nondefault_group_
u1=1
u2=2
diff --git a/tests/install/install-C-selinux.sh b/tests/install/install-C-selinux.sh
index 8513d50b8..ce9e0c360 100755
--- a/tests/install/install-C-selinux.sh
+++ b/tests/install/install-C-selinux.sh
@@ -19,7 +19,7 @@
. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
print_ver_ ginstall
require_selinux_
-
+skip_if_nondefault_group_
echo test > a || framework_failure_
chcon -u system_u a || skip_ "chcon doesn't work"
diff --git a/tests/install/install-C.sh b/tests/install/install-C.sh
index 7d3582c40..6051d57f8 100755
--- a/tests/install/install-C.sh
+++ b/tests/install/install-C.sh
@@ -19,6 +19,31 @@
. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
print_ver_ ginstall
skip_if_setgid_
+skip_if_nondefault_group_
+
+# Note if a group is not specified, install(1) will assume that a file
+# would be installed with the current user's group ID, and thus if the
+# file is the same except that it does have a different group due to
+# its parent directory being g+s for example, then the copy will be
+# done again redundantly in a futile attempt to reset the group ID to
+# that of the current user.
+#
+# install -d -g wheel -m 2775 test # Create setgid dir
+# touch test/a # Create source
+# install -Cv -m664 test/a test/i1 # install source with mode
+# install -Cv -m664 test/i1 test/i2 # install dest
+# install -Cv -m664 test/i1 test/i2 # again to see redundant install
+#
+# Similarly if an existing file exists that is the same and has the
+# current users group ID, but when an actual install of the file would
+# reset to a different group ID due to the directory being g+s for example,
+# then the install will not be done when it should.
+#
+# install -Cv -m664 -g "$(id -nrg)" test/i1 test/i2 # set i2 to uesr's gid
+# install -Cv -m664 test/i1 test/i2 # this should install but doesn't
+#
+# Therefore we skip the test in the presence of setgid dirs
+# An additional complication on HFS is that it...
mode1=0644
mode2=0755