[Libosinfo] [PATCHv4 07/11] Automatically remap OsinfoInstallConfig values if needed

[Zeeshan Ali (Khattak)]

On Tue, Dec 18, 2012 at 3:31 AM, Zeeshan Ali (Khattak)
<zeeshanak at gnome.org> wrote:
> On Mon, Dec 17, 2012 at 11:07 PM, Christophe Fergeau
> <cfergeau at redhat.com> wrote:
>> Now that OsinfoInstallConfig has access to the
>> OsinfoInstallConfigParamList for the OsinfoInstallScript that is
>> being configured,
>
> In the previous patch (06/11) you just added the setter/getter for
> config params to OsinfoInstallConfig but nothing yet sets the params.
> Either I'm totally confused or the ordering of these patches is
> somehow wrong.
>
>> we can use the OsinfoDatamap that is optionally
>> set on a given parameter to automatically translate a value for
>> this parameter from a generic libosinfo value to an OS-specific one.
>
> Assuming config params in OsinfoInstallScript has (or can has) access
> to datamaps, I wonder if there is any need to involve
> OsinfoInstallConfig at all here. The changes will be less intrusive
> that way AFAICT.

Ignore this comment completely please. I should have seen the whole
series before commenting. :(

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124