[libvirt] [libvirt-glib 14/37] Implement gvir_config_domain_set_os

Christophe Fergeau cfergeau at redhat.com
Tue Nov 15 14:29:57 UTC 2011


On Fri, Nov 11, 2011 at 04:25:29PM +0100, Marc-André Lureau wrote:
> On Thu, Nov 10, 2011 at 9:33 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> > +/* Must come last since it uses types defined in the other headers */
> > +#include <libvirt-gconfig/libvirt-gconfig-domain.h>
> 
> This is weird, can't libvirt-gconfig/libvirt-gconfig-domain.h include
> libvirt-gconfig/libvirt-gconfig-clock.h (and others) instead?

These headers don't include anything, I wasn't sure this was done on
purpose or not, I chose not to add new includes to these files. I'll do
what you suggest the next time I get include ordering issues in this file,
unless you want this "hack" gone now.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111115/041a3a21/attachment-0001.sig>


More information about the libvir-list mailing list