X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/30fa7724572ae8a5ecbbf22012fa6fe1aea288b4..1618c9e6d179fc46285cf556c5bb090ed0e9e503:/runtests.sh diff --git a/runtests.sh b/runtests.sh index 85030d0b..558d986c 100755 --- a/runtests.sh +++ b/runtests.sh @@ -85,7 +85,7 @@ # they're explicitly given on the command line. # Also, we can't count on 'cp -a' or 'mkdir -p', although they're -# pretty handy. +# pretty handy (see function makepath for the latter). # I think some of the GNU documentation suggests that we shouldn't # rely on shell functions. However, the Bash manual seems to say that @@ -100,6 +100,8 @@ # You cannot do "export VAR=VALUE" all on one line; the export must be # separate from the assignment. (SCO SysV) +# Don't rely on grep -q, as that doesn't work everywhere -- just redirect +# stdout to /dev/null to keep it quiet. # STILL TO DO: @@ -124,6 +126,7 @@ set -e . "./shconfig" RUNSHFLAGS='-e' +export RUNSHFLAGS # for Solaris [ -d /usr/xpg4/bin ] && PATH="/usr/xpg4/bin/:$PATH" @@ -144,8 +147,10 @@ if [ -f /usr/bin/whoami ]; then testuser=`/usr/bin/whoami` elif [ -f /usr/ucb/whoami ]; then testuser=`/usr/ucb/whoami` +elif [ -f /bin/whoami ]; then + testuser=`/bin/whoami` else - testuser=`id -un || whoami || echo ${LOGNAME:-${USERNAME:-${USER:-'UNKNOWN'}}}` + testuser=`id -un 2>/dev/null || echo ${LOGNAME:-${USERNAME:-${USER:-'UNKNOWN'}}}` fi echo " testuser=$testuser" @@ -158,6 +163,12 @@ else echo " preserve_scratch=no" fi +# We'll use setfacl if it's around and it supports the -k option. +if setfacl --help 2>/dev/null | grep ' -k,' >/dev/null; then + setfacl=setfacl +else + setfacl=true +fi if [ ! -f "$rsync_bin" ]; then echo "rsync_bin $rsync_bin is not a file" >&2 @@ -192,6 +203,9 @@ export scratchdir suitedir prep_scratch() { [ -d "$scratchdir" ] && rm -rf "$scratchdir" mkdir "$scratchdir" + # Get rid of default ACLs and dir-setgid to avoid confusing some tests. + $setfacl -k "$scratchdir" + chmod g-s "$scratchdir" return 0 }