On 09/21/2015 06:12 PM, Raphael Hertzog wrote: > Hi, > > On Sun, 20 Sep 2015, Joerg Jaspert wrote: >>> (It seems they are implicitly considered a master when updating the >>> bikeshed's ACLs) I would have thought "Owner:" would make more sense >>> than "Master:" fwiw. >> >> Then you end up having multiple owners. Master IMO shows better what the >> intended value of it is - the ones with access to change around its >> settings. (Got to that from the MASTER role in irc.debian.orgs chanserv). > > FWIW, I also find Owner clearer than Master. Or Admin. But master is > rather unusual. > >> Your -pull-from is basically my -add, just with an added suite. My way >> of thinking is coming from a way more limited usage, so I adjusted add >> to optionally take Suite as parameter, now one can select from where the >> package comes. > > BTW bikeshed-add and bikeshed-rm should be named "bikeshed-add-package" > and "bikeshed-rm-package" otherwise they end up at the same level > than "bikeshed-modify" and you would expect the current commands to > add/remove a bikeshed instead of actually adding/removing a package > in it. > > Cheers,
In which case maybe we want bikeshed-pkg-{add,rm} (and probably more later, like maybe update or sync (from another bikeshed / suite)?). Just my 2 cents... Thomas