Thanks Sundar!
Sundar Yamunachari wrote:
> Evan,
>
> I will look at installadm changes. If it goes faster then I will look
> at the rest of the code changes.
>
> - Sundar
>
> Evan Layton wrote:
>> My apologies for the delay. I have fixed the problems with the
>> mis-merge, sanity tested the changes and updated the webrev. I've
>> listed out the files and split things up between the library and
>> installadm. Just pick a subset of files and let me know what you're
>> looking at som we make sure everyting gets covered.
>>
>> And thanks again to Sue for catching the mis-merge!!!
>>
>> Thanks!
>> -evan
>>
>> installadm files:
>> usr/src/cmd/installadm/Makefile
>> usr/src/cmd/installadm/installadm-common.sh
>> usr/src/cmd/installadm/installadm.c
>> usr/src/cmd/installadm/installadm.h
>> usr/src/cmd/installadm/installadm_util.c
>> usr/src/cmd/installadm/server.xml
>> usr/src/cmd/installadm/setup-service.sh
>> usr/src/cmd/installadm/svc-install-server
>> usr/src/pkgdefs/SUNWinstall-libs/prototype_com
>>
>> libaiscf Library files:
>> usr/src/lib/Makefile
>> usr/src/lib/libaiscf/Makefile
>> usr/src/lib/libaiscf/ai_create.c
>> usr/src/lib/libaiscf/ai_trans.c
>> usr/src/lib/libaiscf/ai_utils.c
>> usr/src/lib/libaiscf/libaiscf.h
>>
>>
>> And if anybody is REALLY board here's the test files...
>>
>> optional files:
>> usr/src/cmd/installadm/test/installadm_test.c
>> usr/src/cmd/installadm/test/manual_tests
>>
>>
>> Evan Layton wrote:
>>> Sue pointed out that the webrev shows that I have a mis-merge so if
>>> you're looking at the changes please stop. I'll get the mis-merge
>>> fixed and send out a new webrev as soon as possible.
>>>
>>> Thanks,
>>> -evan
>>>
>>>
>>> Evan Layton wrote:
>>>>
>>>> We need to get the fix for 7218 code reviewed. There are around 2000
>>>> lines of changes and new code so we'll need volunteers that can take
>>>> sections of the code to look at. I'm sending this out now so the
>>>> link is out there but I'll want to get things divided up tomorrow
>>>> morning and get folks looking at this tomorrow.
>>>>
>>>> The bug is 7218
>>>> http://defect.opensolaris.org/bz/show_bug.cgi?id=7218
>>>>
>>>> Webrev:
>>>> http://cr.opensolaris.org/~evanl/7218/
>>>>
>>>> Thanks!
>>>> -evan
>>>> _______________________________________________
>>>> caiman-discuss mailing list
>>>> caiman-discuss at opensolaris.org
>>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>>
>>> _______________________________________________
>>> caiman-discuss mailing list
>>> caiman-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>
>> _______________________________________________
>> caiman-discuss mailing list
>> caiman-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>
