jan damborsky wrote: > Hi Sundar, > > the changes look good to me. > I have only question about bugs 3968, 3945 - I don't > see any specific changes in the webrev which might be > related to those issues ? Are they already implemented ? Thanks Jan.
3968: installadm does not create directory The directory is created in setup-image.sh. I just removed lines 347-365 from installadm.c to fix this problem. 3945:installadm delete-service doesn't remove the target directories This happened because the code in installadm.c: remove_service_data() is not returning directory correctly (line 1289 specifically) - Sundar > > Thank you, > Jan > > > Sundar Yamunachari wrote: >> Hi Jan and Sue: >> >> Please review the code for bug fixes. >> >> The webrev is at: http://cr.opensolaris.org/~ysundar/3976/ >> >> 3976 AI DHCP lease is short and not being extended >> http://defect.opensolaris.org/bz/show_bug.cgi?id=3976 >> >> 4070 Client booted with AI image cannot access outside the subnet >> http://defect.opensolaris.org/bz/show_bug.cgi?id=4070 >> >> 3968 installadm does not create directory >> http://defect.opensolaris.org/bz/show_bug.cgi?id=3968 >> >> 3945 installadm delete-service doesn't remove the target directories >> http://defect.opensolaris.org/bz/show_bug.cgi?id=3945 >> >> Thanks, >> Sundar >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >
