Thank you Drew and John!
Sue

On 09/23/11 09:41, John Fischer wrote:
LGTM2

John

On 09/23/11 09:27 AM, Drew Fisher wrote:
LGTM!

-Drew

On 9/23/11 10:18 AM, Sue Sohn wrote:
Yes, that is better, thanks.

Updated webrev:
https://cr.opensolaris.org/action/browse/caiman/sohn/7094044_2/webrev.7094044.2

Sue

On 09/23/11 09:06, Drew Fisher wrote:
Sue,

Instead of using /usr/bin/touch, you can just do:

with open(os.path.join(self.app.tmp_dir, 'lic_OTN'), "w"):
pass

If you do it this way, you don't have to import solaris_install.run

-Drew



On 9/23/11 9:41 AM, Sue Sohn wrote:
Can I get a review of the changes for:

7094044 slim_install_unit test failure after 7093245 is integrated

Webrev:
https://cr.opensolaris.org/action/browse/caiman/sohn/7094044/webrev.7094044

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


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

Reply via email to