Updated webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/7156990-v2
It adds a test case for revert and fixes that issue as well.
Brock
On 03/29/12 12:52, Brock Pytlik wrote:
On 03/29/12 09:58, Shawn Walker wrote:
On 03/29/12 09:22, Bart Smaalders wrote:
On 03/27/12 15:42, Brock Pytlik wrote:
Webrev:
https://cr.opensolaris.org/action/browse/pkg/bpytlik/7156990-v1
Bug:
7156990 setting a mediator with signed packages containing variants
doesn't work
This is causing some real pain, so if we could get this reviewed
quickly, that'd be great.
This appears correct. Is the same bug present in the revert code?
plan_revert() does appear to have the same issue; verify() and
repair() seem to be correct.
Ah, I hadn't thought of revert. I'll fix and add a test case there.
t_pkgsign.py also needs a copyright update.
Otherwise, looks right to me as well.
Thanks for the reviews,
Brock
-Shawn
_______________________________________________
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