LGTM.

--Karen

On 07/12/12 02:16 PM, Susan Sohn wrote:
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

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to