Sue: LGTM.
----- Original Message ----- From: [email protected] To: [email protected], [email protected] Cc: [email protected] Sent: Thursday, July 12, 2012 2:21:48 PM GMT -08:00 US/Canada Pacific Subject: Re: [caiman-discuss] code review request for 7183729 Mary and John, Thanks for the review. I realized that I also needed to modify the comments preceding the change, so a re-review is requested. Same location. Sue On 07/12/12 01:29 PM, John Fischer wrote: > LGTM. > > John > > On 07/12/12 12:58 PM, Susan Sohn wrote: >> Can I get a couple of reviews of the changes for U1 bug: >> >> 7183729 Modify S11.1 default AI manifest to install entire branch 0.175.1 >> rather than entire@latest >> http://monaco.us.oracle.com/detail.jsf?cr=7183729 >> >> Webrev: >> https://cr.opensolaris.org/action/browse/caiman/sohn/7183729/webrev.7183729 >> >> Testing: >> Created service with new default manifest. >> Installed, pointing to repository containing Update 1, and verified build 20 >> was successfully >> installed. >> Installed, pointing to repository with only Solaris 11 FCS, and verified >> that install failed with: >> PlanCreationException: The following pattern(s) did not match any allowable >> packages. Try >> using a different matching pattern, or refreshing publisher information: >> pkg:/[email protected] >> No new problems with unit tests >> >> Thanks, >> Sue >> >> >> _______________________________________________ >> 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
