Hi Alok,

Alok Aggarwal wrote:
> Hi Sundar,
>
> On Fri, 10 Oct 2008, Sundar Yamunachari wrote:
>
>> *ai_x86_image.xml:
>>
>> **    *The path names of the scripts like  ai_pre_bootroot_pkg_image_mod, 
>> ai_post_bootroot_configure, ai_post_bootroot_archive are specific to your 
>> directory (pointing to /brmnas/aalok/slim_source-ai/...   ). Should they be 
>> pointing to usr/src/...instead?
>>
>>   I see only ai_post_bootroot_configure, and ai_post_bootroot_archive are 
>> listed in the code review. What about ai_pre_bootroot_pkg_image_mod?
>
> Thanks for the review, please check out -
>
> http://cr.opensolaris.org/~aalok/3839-behave/

When playing with those scripts during process of building
AI image, I noticed that 'ai_post_bootroot_configure'
complained about '/platform/i86pc/multiboot' missing.
Looking at that script, there is probably a typo:

84 if [ ! -d ${BR_BUILD}/platform/i86pc/multiboot ] ; then
->
84 if [ ! -f ${BR_BUILD}/platform/i86pc/multiboot ] ; then

Thank you,
Jan


Reply via email to