Hi Alok,
Alok Aggarwal wrote:
> Hi Jan,
>
> On Thu, 12 Nov 2009, Jan Damborsky wrote:
>
>>> installadm-common.sh: lines 463-464: Sorry if I'm dense but
>>> I don't see why this check is needed. If x86.microroot exists,
>>> it also means that split 32/64 bit archives don't exist; so
>>> we should pick up the boot archive from /${BootLofs}/x86.microroot.
>>>
>>> Otherwise, pick it up from the appropriate platform directory.
>>> Which cases would this miss?
>>
>> You are right. Too much 'ifs' as Jean also pointed out :-)
>> I have removed one of them and added comment.
>>
>> The webrev has been updated accordingly.
>
> Looks good.
Thanks a lot !
Jan
