On 04/16/2013 07:41 AM, Osier Yang wrote: > Which is already included in "internal.h", later patch will add > syntax-check to avoid it. > --- > src/remote/remote_driver.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/src/remote/remote_driver.h b/src/remote/remote_driver.h > index 7288381..bd907c0 100644 > --- a/src/remote/remote_driver.h > +++ b/src/remote/remote_driver.h > @@ -24,8 +24,6 @@ > #ifndef __VIR_REMOTE_INTERNAL_H__ > # define __VIR_REMOTE_INTERNAL_H__ > > -# include "libvirt/virterror.h" > - > # include "configmake.h" Except that none of the remaining includes pull in internal.h, so this change would make the header not self-contained. I'd be happier if you did s#libvirt/virterror.h#internal.h# instead of deleting the line. ACK with that change. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list