The testsuite had several uses of `set - $list', such that $list could safely begin with `-'. But 1) POSIX has obsoleted this usage, and 2) in several shells (at least ash and bash), using - instead of -- as the separator gives the shell the right to reset -v and -x, which kills the trace when VERBOSE=yes. In all four cases patched below, it is easily verifiable that the expanded arguments at those points cannot start with -, so a separator is not needed since the expansion will not be confused with options; but if you prefer to use the autoconf approach instead of my patch, rewrite each offending line as `set x $list; shift'.
The other portability fix is that on cygwin, mkdir(2) executed from a directory
that has ACLs propagates ACLs to the new directory (as a byproduct of how
Windows NTFS permissions are mapped to POSIX semantics). There are several
other tests in the testsuite that are not tolerant of ACLs, but at least
mkdir/perm can be fixed now as long as we are fixing `set -'.
$ ls -ld .
drwxrwxrwx+ 16 Administrators Domain Users 12288 Apr 26 10:42 ./
$ mkdir dir
$ ls -ld dir
drwxr-xr-x+ 2 eblake Domain Users 0 Apr 26 10:42 dir/
$ getfacl dir
# file: dir
# owner: eblake
# group: Domain Users
user::rwx
group::r-x
mask:rwx
other:r-x
default:user::rwx
default:group::r-x
default:other:r-x
2005-04-26 Eric Blake <[EMAIL PROTECTED]> (tiny change)
* tests/misc/nice (tests): Don't use `set -', it resets -x.
* tests/mv/part-hardlink (fail): Likewise.
* tests/stty/row-col-1 (tests): Likewise.
* tests/mkdir/perm (tests): Likewise. Also, allow for default
directory ACLs.
--
Eric Blake
coreutils.patch
Description: Binary data
_______________________________________________ Bug-coreutils mailing list [email protected] http://lists.gnu.org/mailman/listinfo/bug-coreutils
