tags 619726 pending thanks On Sat, Mar 26, 2011 at 03:21:09PM +0100, Iustin Pop wrote: > /etc/cron.daily/man-db has this snippet: > > # expunge old catman pages which have not been read in a week > if [ -d /var/cache/man ]; then > cd / > if ! dpkg-statoverride --list /var/cache/man >/dev/null 2>&1; then > chown -R man /var/cache/man || true # just in case > fi > … > > The chown bit there will needlessly set the owner to man, even if all > the files already have the correct owner. > > Could you instead do this? > > find /var/cache/man ! -owner man -print0 | xargs -r0 chown man || true > > It's a small optimisation, but it would eliminate useless I/Os.
(That should be -user rather than -owner.) Thanks; done in cron.daily and postinst for my next upload. -- Colin Watson [[email protected]] -- To UNSUBSCRIBE, email to [email protected] with a subject of "unsubscribe". Trouble? Contact [email protected]
