Alok,

That looks like a comment waiting to happen.

Jean

Alok Aggarwal wrote:
>
> On Fri, 20 Feb 2009, Jens Deppe wrote:
>
>> Here's the call chain as I see it:
>>
>> auto_perform_install() -> install_from_manifest() -> 
>> create_package_list_file(B_FALSE)
>>
>> In create_package_list_file(), the 'if (hardcode)' block is skipped 
>> and then the code below results in no packages being defined in the 
>> package list file:
>>
>>    package_list = ai_get_manifest_packages(&num_packages);
>>
>>    if (num_packages <= 0)
>>        goto errorout;
>>
>> I would think that instead of bailing at this point the default list 
>> of packages should be defined for installation. No?
>
> The missing piece here is that ai_get_manifest_packages()
> will either return the list of packages if one is specified
> in the manifest or return the default list (as specified in
> ai_manifest.defval.xml).
>
> Either way this call will give you a list of packages unless
> the ai_manifest.defval.xml has been changed to have an empty
> list of default packages.
>
> Alok
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss


Reply via email to