Just one quick comment:
327 says: "if more than one IP is configured" but 329 uses != 1.
Shouldn't that be > 1 ?
Otherwise looks fine.
-Drew
On 7/14/11 2:58 PM, Jesse Butler wrote:
Could I please get a review of my change for:
7066592 <http://monaco.us.oracle.com/detail.jsf?cr=7066592> $serverIP
can't be used for S11 Express images
I mistakenly made a change in 170 that sets $serverIP for all install
images, but this cannot be done for Solaris 11 Express images because
it doesn't have support for this functionality. I actually didn't know
at the time we were shipping with support for S11 Express images, so I
didn't even test it.
Now, I have. Here's the webrev, which I've already had one person take
a peek at.
https://cr.opensolaris.org/action/browse/caiman/jesseb/7066592/webrev/
Thanks
Jesse
_______________________________________________
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
