Nirmal,
Looks fine.
John
On 05/30/12 12:33 AM, Nirmal Agarwal wrote:
Hi John
Thanks for the review.
On 05/29/12 22:19, John Fischer wrote:
Nirmal,
The fix looks good to me. Are there any additions to the slim_test that
are needed to catch this issue?
I don't think so. Please let me know if you think otherwise.
Also, I have fixed issue pointed by Sue. Please find the latest webrev :
webrev -diff :
https://cr.opensolaris.org/action/browse/caiman/nirmal27/7170315-diff/webrev-diff/
webrev:
https://cr.opensolaris.org/action/browse/caiman/nirmal27/7170315-2/webrev/
Thanks,
Nirmal
Thanks,
John
On 05/29/12 08:15 AM, Nirmal Agarwal wrote:
Hi all
Can I please get 2 reviewer for this fix.
Thanks
Nirmal
On 5/25/2012 7:50 PM, Nirmal Agarwal wrote:
Hi all
Can I please get code review for CR 7170315 .
7170315 webservd lost permission to validate profile
webrev :
https://cr.opensolaris.org/action/browse/caiman/nirmal27/7170315/webrev/
Testing :
--> Ran installadm validate -P without authorization and it works as
expected.
Slim Test: Pass
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
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
