Re: [PATCH] shared repository settings enhancement.
- From
Junio C Hamano <junkio@cox.net>
- Date
- Jun 10, 2006, 04:19 UTC
- Message-ID
- <7vver9k5gg.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <Pine.LNX.4.64.0606092103170.5498@g5.osdl.org>
Linus Torvalds <torvalds@osdl.org> writes:
Show 19 quoted lines
> How about making it be
>
> [core]
> sharedrepository = {umask | user | group | everybody}
>
> and allow the old boolean expression syntax to mean "0/false means umask,
> 1/true means group".
>
> So you'd have:
>
> - umask/0/false means "use 0777 permissions with default umask"
> - user means "use 0500 permissions"
> - group means "use 0550 permissions"
> - everybody means "use 0555 permissions"
>
> (where "5" is r-x, and only for directories, and obviously degenerates to
> just "4" aka r-- for regular files).
>
> That sounds really pretty self-explanatory and obvious, wouldn't you say?Yes, the user can mistype "gruop", people would start making noises about having "world" as a synonym for "everybody", and the parsing becomes somewhat cumbersome, and all that trouble, but on the other hand that is probably the easiest to explain.