[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [virt-tools-list] [libosinfo PATCHv2] add datamap support



On Wed, Dec 12, 2012 at 03:44:06AM +0200, Zeeshan Ali (Khattak) wrote:
> On Tue, Dec 11, 2012 at 8:20 PM, Christophe Fergeau <cfergeau redhat com> wrote:
> > Hey,
> 
> Hi,
> 
> > Here is a v2 of my datamap patches which addresses the review comments.
> > I've also moved the new symbols to a new section in libosinfo.syms as there
> > has been a release since the initial posting
> >
> > Changes since v1:
> > - rename osinfo_install_script_get_config_paramlist to
> >   osinfo_install_script_get_config_params
> > - fix wrong API doc mentioning g_list_free
> > - add comments in patch 7 about 'name' and 'id' being assumed to be the
> >   same in some functions
> 
> If that is all the changes made in v2, then ACK for the series.

Hmmm I can't promise these are the only changes, these are the most
prominent ones I remembered when sending this email, but maybe I've
forgotten a few items.

Christophe

Attachment: pgpo8hk5LKqJp.pgp
Description: PGP signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]