On 01-11-14 16:37, Paul Gevers wrote: > Furthermore, I have been reading up on the use of cp. It seems, but if > you are using ACL's I like you to test, that cp opens the file for > writing, so it should keep existing ACL's AFAICT.
My latest patch works correctly with acl: (pbuild29710)root@wollumbin:/# setfacl -m u:paul:5 /etc/roundcube/debian-db.php (pbuild29710)root@wollumbin:/# getfacl /etc/roundcube/debian-db.php getfacl: Removing leading '/' from absolute path names # file: etc/roundcube/debian-db.php # owner: root # group: www-data user::rw- user:paul:r-x group::r-- mask::r-x other::--- (pbuild29710)root@wollumbin:/# dpkg-reconfigure roundcube-core dbconfig-common: writing config to /etc/dbconfig-common/roundcube.conf + [ /etc/roundcube/debian-db.php ] + ls -al /tmp/dbconfig-generate-include.l6EBqg -rw-r-x---+ 1 root www-data 546 Nov 1 18:46 /tmp/dbconfig-generate-include.l6EBqg + ls -al /etc/roundcube/debian-db.php -rw-r-x---+ 1 root www-data 546 Nov 1 16:23 /etc/roundcube/debian-db.php + [ 1 ] + ucf --debconf-ok /tmp/dbconfig-generate-include.l6EBqg /etc/roundcube/debian-db.php Replacing config file /etc/roundcube/debian-db.php with new version + rm -f /tmp/dbconfig-generate-include.l6EBqg + ls -al /etc/roundcube/debian-db.php -rw-r-x---+ 1 root www-data 546 Nov 1 18:46 /etc/roundcube/debian-db.php + check_permissions + local line + dpkg-statoverride --list /etc/roundcube/debian-db.php + [ ] + [ ] + ls -al /etc/roundcube/debian-db.php -rw-r-x---+ 1 root www-data 546 Nov 1 18:46 /etc/roundcube/debian-db.php + set +x apache2_invoke roundcube.conf: already enabled invoke-rc.d: policy-rc.d denied execution of reload. invoke-rc.d: policy-rc.d denied execution of reload. (pbuild29710)root@wollumbin:/# getfacl /etc/roundcube/debian-db.php getfacl: Removing leading '/' from absolute path names # file: etc/roundcube/debian-db.php # owner: root # group: www-data user::rw- user:paul:r-x group::r-- mask::r-x other::--- (pbuild29710)root@wollumbin:/# Paul
signature.asc
Description: OpenPGP digital signature