I'm sorry I didn't have a chance to review this. Your use of a temporary fmri in the last hunk is unnecessary, and will slow the operation down, at least a little bit. You should create the fmri, retrieve the authority, and call set_authority() on the original fmri object. That'll save us time on object creation and parsing, as well as generally looking less messy.
A new bug to cover this will be fine. Thanks, Danek _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
