Looks good to me now!
Thanks,
Jack
On 11/03/08 12:02, Karen Tung wrote:
> Hi Jack,
>
> Thanks for the quick review.
>
> Yes, it needs to be added to the Makefile too.
> The webrev is updated with that change:
>
> http://cr.opensolaris.org/~ktung/4478/
>
> Thanks,
>
> --Karen
>
> Jack Schwartz wrote:
>> Hi Karen.
>>
>> pre_bootroot_pkg_image_mod needs to be added to the
>> distro_const/utils/Makefile also.
>>
>> Otherwise looks fine.
>>
>> Thanks,
>> Jack
>>
>> On 11/03/08 11:56, Karen Tung wrote:
>>> Hi,
>>>
>>> Can I get 2 reviewers for the fix for 4478 asap?
>>>
>>> 4478 ai_x86_image.xml and slim_cd.xml looking for nonexistent file
>>>
>>> http://cr.opensolaris.org/~ktung/4478/
>>>
>>> Thanks,
>>>
>>> --Karen
>>>
>>> _______________________________________________
>>> caiman-discuss mailing list
>>> caiman-discuss at opensolaris.org
>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
>>>
>>
>