On 12/15/10 11:43 AM, Jack Schwartz wrote:
 Hi Drew.

Looks good modulo the other's comments, and one nit:

install_target/ti.py: 114:

Adding the third arg of None doesn't do anything, since the third arg defaults to None anyway. (I'm looking at install_target/zpool.py for this.) How come this line was changed? How come line 117 wasn't changed as well, as the two lines are now inconsistent with each other in this respect?

Good catch.  Everything's in place now.


Thanks Jack!

-Drew


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

Reply via email to