On 04/25/12 17:48, Brock Pytlik wrote:
Webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/7160289-v1
Bugs:
7160289 pkgrecv failing to pull build s11u1_13: pkgrecv: 'add_file' failed.
7164129 pkgrecv sets republish goal incorrectly when transferring packages
This fixes an issue where chain certificate files for signature actions
weren't always where pkgrecv expected to be if a file for the signature
payload was present. It also adds checks to the progress checker so that
when republishing is finished, we make sure we moved the amount of files
and bytes that we expected to. Had those assertions been there, bug
7164129 wouldn't have happened.
src/modules/client/progress.py:
line 369: I think 'assert_on_mismatch' or 'assert_mismatch' would be
a better parameter than 'merge'. 'merge' seems too specific. May
not matter in the long-run with Dan's upcoming rewhack.
src/modules/client/transport/transport.py:
line 2313: can you update docstring explaining purpose of in_hash?
line 3022: needs wrapping
-Shawn
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
