group names with spaces

[Bruno Haible]

Hi,

Building coreutils-6.12.29-a16be on the td152.testdrive.hp.com machine
yields a test failure for me. Log attached below. In short, the problem is
that GNU chown fails to parse a USER:GROUP specification if the GROUP
contains spaces.

$ id
uid=90719(bhaible2) gid=513(Domain Users) groups=513(Domain Users)
$ ls -ld .
drwxr-xr-x  9 bhaible2  Domain Users  4096 Jun 10 19:29 .
$ touch foo
$ src/chown 'bhaible2:Domain Users' foo
src/chown: invalid group: `bhaible2:Domain Users'

Bruno


===========================================
 1 of 344 tests failed
 (51 tests were not run)
 See tests/test-suite.log
 Please report it to address@hidden
===========================================

=======================================================
   GNU coreutils 6.12.29-a16be: tests/test-suite.log   
=======================================================

1 of 344 tests failed.  (51 tests were not run).  

.. contents:: :depth: 2


FAIL: chown/separator.log (exit: 1)
===================================

+ chown --version
chown (GNU coreutils) 6.12.29-a16be
Copyright (C) 2008 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by David MacKenzie and Jim Meyering.
+ . ../../tests/test-lib.sh
++ unset function_test
++ eval 'function_test() { return 11; }; function_test'
+++ function_test
+++ return 11
++ test 11 '!=' 11
+++ pwd
++ test_dir_=/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests
+++ this_test_
+++ echo ././../../tests/chown/separator
+++ sed 's,.*/,,'
++ this_test=separator
+++ /house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/src/mktemp -d 
--tmp=/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests 
cu-separator.XXXXXXXXXX
++ 
t_=/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT
++ trap 'st=$?; cleanup_; 
d=/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT;
    cd /house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests && chmod -R 
u+rwx "$d" && rm -rf "$d" && exit $st' 0
++ trap '(exit $?); exit $?' 1 2 13 15
++ cd 
/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT
++ diff --version
++ grep GNU
++ id -u
+ id_u=90719
+ test -n 90719
++ id -un
+ id_un=bhaible2
+ test -n bhaible2
++ id -g
+ id_g=513
+ test -n 513
++ id -gn
+ id_gn='Domain Users'
+ test -n 'Domain Users'
+ fail=0
+ chown '' .
+ for u in '$id_u' '"$id_un"' ''\'''\'''
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 90719:513 .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 90719.513 .
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown '90719:Domain Users' .
chown: invalid group: `90719:Domain Users'
+ fail=1
+ for sep in '$seps'
+ case $u$sep$g in
+ chown '90719.Domain Users' .
chown: invalid user: `90719.Domain Users'
+ fail=1
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 90719: .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 90719. .
+ for u in '$id_u' '"$id_un"' ''\'''\'''
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown bhaible2:513 .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown bhaible2.513 .
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 'bhaible2:Domain Users' .
chown: invalid group: `bhaible2:Domain Users'
+ fail=1
+ for sep in '$seps'
+ case $u$sep$g in
+ chown 'bhaible2.Domain Users' .
chown: invalid user: `bhaible2.Domain Users'
+ fail=1
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown bhaible2: .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown bhaible2. .
+ for u in '$id_u' '"$id_un"' ''\'''\'''
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown :513 .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown .513 .
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown ':Domain Users' .
chown: invalid group: `:Domain Users'
+ fail=1
+ for sep in '$seps'
+ case $u$sep$g in
+ chown '.Domain Users' .
chown: invalid user: `.Domain Users'
+ fail=1
+ for g in '$id_g' '"$id_gn"' ''\'''\'''
+ case $u$g in
+ seps=': .'
+ for sep in '$seps'
+ case $u$sep$g in
+ chown : .
+ for sep in '$seps'
+ case $u$sep$g in
+ chown . .
+ exit 1
+ exit 1
+ st=1
+ cleanup_
+ :
+ 
d=/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT
+ cd /house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests
+ chmod -R u+rwx 
/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT
+ rm -rf 
/house/bhaible2/coreutils-6.12.29-a16be/build-freebsd/tests/cu-separator.Y7qO4qIsJT
+ exit 1
gmake[4]: *** [test-suite.log] Error 1