doh.

I totally overlooked this.   /sigh.

I've updated the webrev here:

https://cr.opensolaris.org/action/browse/caiman/drewfish/7190917_2/webrev/

Thanks, Karen, for covering me.  :)

-Drew

On 8/14/12 3:12 PM, Karen Tung wrote:
Hi Drew,

Lines 350-354 of the new file has checks for lock file. Now that you move the check higher,
should those lines be removed?

Also, please update the bug with the reason for needing to moving the lines up as we discussed on IM.

Thanks,

--Karen

On 08/14/12 12:35 PM, Drew Fisher wrote:
Good afternoon!

Could I please get a few sets of eyes on a fix for:

7190917 <http://monaco.us.oracle.com/detail.jsf?cr=7190917> distro-const building was interrupted by another dc building operation

https://cr.opensolaris.org/action/browse/caiman/drewfish/7190917/webrev/

I've verified a second run of DC will no longer trash the first run. I've also run pep8 and pylint and found no new regressions.

DC unittests are clean as well.

Thanks!

-Drew





_______________________________________________
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