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