Ok. It should be trivial to add --add-mirror --remove-mirror as longopts for -m and -M. I'll go ahead and do that.
-j On Mon, Aug 04, 2008 at 12:39:17PM -0700, Stephen Hahn wrote: > * [EMAIL PROTECTED] <[EMAIL PROTECTED]> [2008-08-04 19:23]: > > > > http://cr.opensolaris.org/~johansen/pkg-mirror1/ > > > > > > http://cr.opensolaris.org/~johansen/pkg-mirror1/src/client.py.wdiff.html > > > ========== > > > 105 + [-O origin_url] [-m mirror to add] [-M mirror to > > > remove] authority > > > > > > I don't think -m and -M are very obvious -- they don't say to me as a > > > user, "add and remove." Specifically, it would be too easy to say -M > > > when I met -m or vice-versa. > > > > > > Could we use --add-mirror and --remove-mirror instead or some other set > > > of letters like -a and -r? > > > > I had a completely separate set of commands that you could use to add or > > remove a mirror. I had run the idea by Dan and Danek, and they were > > fine with it. However, Stephen objected to that usage and wanted these > > to be a subcommand of set-authority. I don't agree, but he was pretty > > firm about that. You'll need to take this up with him. > > set-authority --add-mirror url authority > set-authority --remove-mirror url authority > > seem fine to me. I think that discouraging the introduction of > mirrors as a top-level configurable object on the client is still > sensible, given that we haven't reached a conclusion about multiple > authorities inside a single repository. > > - Stephen > > _______________________________________________ > pkg-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/pkg-discuss _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
