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

Revision history for this message
Parth Malwankar (parthm) 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.

Filed bug #524306 to track this.

« Back to merge proposal