Hi Alok,

Alok Aggarwal wrote:
> Hi Jan,
>
> The only thing we'd want to add to this is
> the case that Sundar asks about.
>
> If either of ai_manifest.rng or ai_manifest.defval.xml
> aren't present in /usr/share/auto_install, there
> really is something wrong with the netinstall image.
> Since it will eventually result in the XML validator
> not being able to validate the AI manifest anyway, we
> should error out even before invoking the XML validator.

I wasn't sure about this - thank you very much for
clarification. I have modified the code and published
updated webrev - it is available at the same location.
Please let me know, if the solution is correct.

Thank you very much for reviewing this,
Jan


>
> Alok
>
> On Thu, 9 Oct 2008, jan damborsky wrote:
>
>> Hi Alok,
>>
>> could you please review easy fix for following AI bug ?
>>
>> 3805 AI Service choosing engine should store obtained AI manifest to 
>> '/tmp/ai_manifest.xml'
>>
>> * bug
>> http://defect.opensolaris.org/bz/show_bug.cgi?id=3805
>>
>> * Webrev:
>> http://cr.opensolaris.org/~dambi/bug-3805/
>>
>>
>> Thank you very much,
>> Jan
>>
>>
>>


Reply via email to