On Sat, Dec 29, 2007 at 02:37:58PM -0500, David Roundy wrote: > On Sun, Dec 23, 2007 at 02:34:40PM +0000, Ian Lynagh wrote: > > On Sun, Dec 23, 2007 at 08:28:03AM -0500, David Roundy wrote: > > > On Sun, Dec 23, 2007 at 12:15:35PM +0000, Ian Lynagh wrote: > > > > On Thu, Dec 20, 2007 at 07:10:44AM -0500, David Roundy wrote: > > > > > On Thu, Dec 13, 2007 at 06:53:07PM +0000, Ian Lynagh wrote: > > > > > > > > > > This sounds scary to me. It looks like it'd have all the problems > > > > > that setpref has, and I'd rather not repeat that mistake (or at > > > > > least keep using the same implementation of that mistake). > > > > > > > > The only problems with setpref I can think of are caused by the > > > > details of its implementation, not anything fundamental. > > > > > > Yes, the trouble is that we only keep one copy of the prefs, rather than > > > one pristine copy + one local copy. I believe that's what you were > > > proposing to do with this permissions-mapping file. > > > > The pristine copy is _darcs/whatever, and users should never edit this. > > Ah, I had thought that you'd suggested putting this in _darcs/prefs/, since > it was a per-repository property. How were you going to handle the > per-repository permissions file that maps the arbitrarily permission tags > onto actual permisions? It would seem a shame to make all users manage this > by hand.
There can be darcs commands to manipulate it, but users would be allowed to edit it by hand. In the common case you would just leave teh default "normal" and "executable" options alone, though. > > When they want to change it, they "darcs setperm foo bar" and this puts > > something in _darcs/patches_pending. > > This is to change the permissions of a file. You suggested a patch (I > thought) that created a new permissions type. Right, you'd need that as well, and that would also put something in _darcs/patches/pending. Thanks Ian _______________________________________________ darcs-devel mailing list [email protected] http://lists.osuosl.org/mailman/listinfo/darcs-devel
