Dermot,

Looks good to me.

John

On Jun 23, 2011, at 7:30 AM, Dermot McCluskey wrote:

> Hi all,
> 
> Can I get a couple of reviews for this simple 2-line
> fix:
> 
> http://monaco.us.oracle.com/detail.jsf?cr=7058397
> Solaris2 partition name is incorrect when existing partitions are being 
> preserved
> 
> http://cr.opensolaris.org/~dermot/webrev_7058397/
> 
> A value which was an int at some earlier point in development,
> is now a string, and needs to be cast before comparison.
> 
> I have tested by performing an Install with the configuration
> from the Bug, and confirming that I can boot Solaris afterwards.
> 
> Thanks,
> - Dermot
> 
> _______________________________________________
> 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