John Rice wrote: > [email protected] wrote: > >>> The webrev http://cr.opensolaris.org/~padraig/ips-6268-v1/ fixes >>> 6268 PackageManager does not work with latest ips code >>> >>> The change to call to get_manifest is necessary because of changes in >>> changeset 838. >>> The change to TransportFailures is to specify where it comes from. >>> >>> >> If you pass on the TransportFailures exception, you're not reporting any >> failure information to the user. What happens when the manifest isn't >> successfully retrieved due to transient network errors? >> >> > We are seeing some of these and do not want to bug a user if we can't > get a description for a given package, the best place to handle this > would be to log the transient errors, but there is no general logging > framework in place yet. If the user goes and clicks on a package we will > attempt to refetch the information if it is not available, due to such > transient errors. > >> The second thing is that this code is continuing to use the image >> object. >> > We have REF's logged to get API's to allow us to remove all direct > access to the image object, as soon as the API's are available we will > be more than happy to change over to using them. > Actually, I looked at the RFE's that I know of this afternoon, and didn't see this particular issue covered by any of them. I'm aware of 3 (search, list, authority) and didn't see anything in any of them to cover this use. Can you please either point me to the fourth (and others as well if I'm unaware of them) which covers this, or augment one of the existing ones to make it more clear that this is included in the bug.
Brock >> The API was designed so that the GUI wouldn't manipulate the >> image object directly. If you're depending upon behavior in the image >> object, you'll potentially get broken by an unknown number of changes. >> If there's functionality that's missing from the API, you should file a >> bug against the API instead of using the image object directly. >> >> -j >> _______________________________________________ >> 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 > _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
