On Fri, Apr 11, 2008 at 09:48:36PM -0700, Junio C Hamano wrote: > Heikki Orsila <shdl@xxxxxxxxxx> writes: > > > On Fri, Apr 11, 2008 at 05:53:36PM -0700, Junio C Hamano wrote: > > ... > >> For example, you may want to enforce "ug+rw,o=" in a repository. How > >> would you do that? > > > > Isn't that PERM_GROUP? The user always keeps u+rw for oneself. > > My question was about the "o=" part. I did not see you dropping bits for > others in your patch. > > And if your answer is "the user should have xx7 umask", that defeats the > whole point of your patch, as you are trying to dissociate the umask used > by the user for his usual task and enforce particular permission policy > for the repository. You're correct. There are two options, do you have a preference on this matter? 1. current method + mask off others when needed 2. core.sharedRepository=0xxx I think I like option 2 better (preserving backwards compatibility of course). -- Heikki Orsila Barbie's law: heikki.orsila@xxxxxx "Math is hard, let's go shopping!" http://www.iki.fi/shd -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html