Looks good to me Nirmal. Thanks,
Darren. On 07/03/2012 09:16, Nirmal Agarwal wrote: > Hi all > > Can I please get a code review for CR 7125582. > > 7125582 installadm create-client incorrectly generates usage error mesg > for non usage type error > > Webrev : > https://cr.opensolaris.org/action/browse/caiman/nirmal27/7125582/webrev/ > > Slim-test : pass > Source is pep8 clean. > > Test : > > root@server:~# installadm create-client -e 00:14:4f:a7:65:70 -n S11-ai-x86 > Warning: custom network configuration is enabled. Please be sure that > the IP addres > s for server is static. > The specified service does not exist: S11-ai-x86 > > "installadm" command exist without showing usage message when install > service doesn't exist. > > 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
