This change looks good to me. -j
On Mon, Dec 10, 2007 at 02:48:56PM -0500, Richard Lowe wrote: > > Hey all, > > Could you please review: > > #60 'pkg status does_not_exist' throws a traceback > > Bug is: http://defect.opensolaris.org/bz/show_bug.cgi?id=60 > Webrev is: http://cr.opensolaris.org/~richlowe/pkg_60 > > It's somewhat unusually shaped to allow it to fail only partially > (much like a recent change of Danek's), such that: > > % pkg status foo SUNWzsh bar > > Will show the status of SUNWzsh, but warn about the other two (and > exit with failure) > > % pkg status foo SUNWzsh bar > FMRI STATE UFIX > pkg://indiana/[EMAIL PROTECTED] installed ---- > pkg://indiana/[EMAIL PROTECTED] known u--- > pkg: no packages matching 'foo' installed > pkg: no packages matching 'bar' installed > > This causes an extra test to pass (cli-complete is updated > appropriately). > > -- Rich > _______________________________________________ > 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
