Code review comment for lp:~parthm/bzr/2.0_376388_dot_bazaar_ownership

Revision history for this message
Martin Pool (mbp) wrote :

An interesting followon from this would be that we could also do this when creating a .bzr directory.

Also, you should perhaps do the chown and chgrp separately, as the chown may fail when the chgrp can succeed. (Common for non-root users.)

I think overall this is a reasonable change though it might be considered a bit magical.

« Back to merge proposal