Looks good - Kristina
On Mar 14, 2012, at 1:13 AM, Sreedhar Chalamalasetti wrote: > Hi > > Could you please review the following CR fixes > 7153268 - aimanifest get includes trailing space > 7037125 - installadm add-manifest usage/error message is not clear > Webrev: > https://cr.opensolaris.org/action/browse/caiman/sc157166/CR_7153268_7037125/webrev > Testing: > 1. aimanifest: > - aimanifest get ... returning correct value without space at the > end of value. > - aimanifest get -r returning "value path". > 2. installadm add-manifest/create-manifest/update-manifest: Displays error > messages as follows. > - installadm: error: Missing required option -n <service_name>. > - installadm: error: Missing required option -f <manifest_file>. > 3. pep8 clean. > 4. slim_test OK. Results: > /net/indiana-build/export/home/sc157166/bugs/7153268/slim_source/slim_tests_output.txt > > > Thanks > Sreedhar > > > _______________________________________________ > 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
