Hi Nirmal, Looks good to me.
Thanks, Darren. On 09/03/2012 08:53, Nirmal Agarwal wrote: > Hi all > > Can I please get a code review for CR 7125173. > > 7125173 js2ai: error: need more than 1 value to unpack > > Webrev : > https://cr.opensolaris.org/action/browse/caiman/nirmal27/7125173/webrev/ > > Slim-test : pass > Source is pep8 clean. > > Test : > Tested the fix with the following profile : > install_type initial_install > boot_device c1d0 > fdisk c1d0 solaris 40000 > > > Thanks > Nirmal > _______________________________________________ > 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

