Hi Sundar,
Sundar Yamunachari wrote: > 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) I see - thank you for clarifying it ! Jan > > - 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 >> >
