On Wed, Mar 09, 2016 at 05:12:19PM +0100, Fabiano Fidêncio wrote: > On Wed, Mar 9, 2016 at 5:02 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: > > virt-viewer-auth.h does not use any libvirt types, so the #include is > > not needed, and virt-viewer-auth.h is used while building remote-viewer, > > which do not use libvirt compilation flags. This caused failures on > > a freshly installed box without libvirt headers. > > --- > > src/virt-viewer-auth.h | 4 ---- > > 1 file changed, 4 deletions(-) > > > > diff --git a/src/virt-viewer-auth.h b/src/virt-viewer-auth.h > > index 85e3bbc..cdabd00 100644 > > --- a/src/virt-viewer-auth.h > > +++ b/src/virt-viewer-auth.h > > @@ -25,10 +25,6 @@ > > > > #include "config.h" > > > > -#ifdef HAVE_LIBVIRT > > -#include <libvirt/libvirt.h> > > -#endif > > - > > #include "virt-viewer-session.h" > > #include "virt-viewer-util.h" > > > > -- > > 2.5.0 > > > > _______________________________________________ > > virt-tools-list mailing list > > virt-tools-list@xxxxxxxxxx > > https://www.redhat.com/mailman/listinfo/virt-tools-list > > Acked-by: Fabiano Fidêncio <fidencio@xxxxxxxxxx> Thanks, pushed. > > Btw, do you know some automated way to check for not needed includes? ;-) Nope, I don't know. Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
