Brock Pytlik wrote:
Shawn Walker wrote:
Brock Pytlik wrote:
http://cr.opensolaris.org/~bpytlik/ips-12475-v2/
Updated webrev which addresses the issues raised including adding more comments. I've posted it again in case the error messages or documentation still needs tweaking.

So, I think I'd rather see a general 'pkg.depotadm' or 'depotadm' or 'repoadm' utility with the functionality here being an 'upgrade' subcommand. In particular, I believe that would help make it clearer that this program is for a package repository only.

Except it's not for a package repository only. It can be used happily on a client image though the use case is somewhat less compelling.

True, but the primary usage case is currently package repository oriented.

I'd be happy to see this pulled into a depotadm or whatever when that comes along, if it's still needed at that point. For now, I think it belongs in a one-off script.

I'm aware it's a one-off script, but distributing a one-off script in usr/* seems wrong to me. If it is really a one-off, I'd prefer it be placed in util/. I'd rather not see us ship a utility in /usr only to nuke it later in favour of a program that does the exact same thing.

Cheers,
--
Shawn Walker
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to