inherit mode patch

Alexis S. L. Carvalho alexis at cecm.usp.br
Sun Mar 4 12:30:29 CST 2007


Thus spake Matt Mackall:
> Wouldn't it be simpler to just pass mode=x down and be done with it?
> Where mode=None if umask is agreeable? Then opener can simply do:
> 
>   if mode:
>     chmod(f)
> 
> I'd really like to avoid making opener any more complicated than it
> already is.

Fair enough.

> I don't see any useful reason to have just subsets of a repo being
> group-writable. That'll just cause more confusion. So it should really
> be an all or nothing thing based on the permissions of .hg.

When the new layout with store/ was introduced, the idea of having more
than one repo sharing the same store was circulated.  This suggests that
we should get the permissions from the store directory instead of .hg.

Alexis


More information about the Mercurial-devel mailing list