If you do need a reviewer, I think the change obviously fixes the test case.
However it seems that AIPublishPackages.DEFAULT_ARG should have something more sensible than None for "publisher". After all AIPublishPackages.parse_doc() knew to change it from None to "ai-image". Probably better addressed in a separate bug, so you are good. -Dave On Aug 31, 2011, at 12:15 PM, Harold Shaw wrote: > This is a simple one. I neglected to update the unit test for the > distro_const checkpoint, ai_publish_package when I fixed 7083780. Dave, I > don't know if I need to request approval for this putback since it's a unit > test fix. If I do, this is my official request. The bug is: > > 7085313 slim_install unit test falure after 7083780 went in > > Webrev: https://cr.opensolaris.org/action/browse/caiman/hshaw/7085313/webrev/ > > I ran the unit tests to verify that fixed the problem. > > Thanks, > Harold > _______________________________________________ > caiman-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
