Hi everyone, esp. Ethan, Jan and John.

Thanks for your review. Webrev is updated per your comments and other email discussion.

Same location:
http://cr.opensolaris.org/~schwartz/100608.1/webrev/index.html

Note: the two files which were sent for review originally have not changed; only ai_parse_manifest.py and default.xml were added.

Please let me know if you have any further comments.

Tomorrow I will retest AI in light of the changes made, before pushing.

    Thanks,
    Jack


On 06/ 8/10 03:20 PM, Jack Schwartz wrote:
Hi everyone.

Here is a fast code review of a couple of small issues which Driver-Update caused to appear:
16088  <http://defect.opensolaris.org/bz/show_bug.cgi?id=16088>  DDU AI install 
did not copy install_log to system if DDU install fails.
16097  <http://defect.opensolaris.org/bz/show_bug.cgi?id=16097>  AI Driver 
Update gives misleading messages about finding and not-finding packages

Please review:
http://cr.opensolaris.org/~schwartz/100608.1/webrev/index.html

        Thanks,
        Jack

_______________________________________________
caiman-discuss mailing list
caiman-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to