Shawn Walker wrote:
Greetings,

The following webrev contains fixes for the following issues:

8102 pkg.client.api info reports size of zero for packages with cached manifests

webrev:
http://cr.opensolaris.org/~swalker/pkg-8102/

This is happening regardless of architecture and is a result of the CachedManifest work that was recently done, and so is broken in the API and not in any specific client.

The fix was to add a get_size method to Manifest/CachedManifest that takes a list of excludes and returns the size in bytes for the data payloads of the resulting action set and remove the obsolete size attribute from the Manifest classes. The public client API and internal APIs have been changed to use this new function.

Finally, the t_api_info unit tests were updated to ensure that this doesn't happen again.

Cheers,

This looks fine... thanks for fixing this.

- Bart


--
Bart Smaalders                  Solaris Kernel Performance
[email protected]         http://blogs.sun.com/barts
"You will contribute more with mercurial than with thunderbird."
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to