Re: [PATCH] tests: link virpcimock with utils

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

 



On Mon, Apr 10, 2017 at 01:00:58PM +0200, Michal Privoznik wrote:
> On 04/10/2017 09:52 AM, Pavel Hrdina wrote:
> > In virpcimock we use some functions from utils so we should link
> > utils library to provide required symbols.
> > 
> > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> > ---
> >  tests/Makefile.am | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tests/Makefile.am b/tests/Makefile.am
> > index 279e9b7da8..8b9bba31ce 100644
> > --- a/tests/Makefile.am
> > +++ b/tests/Makefile.am
> > @@ -1131,7 +1131,8 @@ virpcimock_la_SOURCES = \
> >  	virpcimock.c
> >  virpcimock_la_CFLAGS = $(AM_CFLAGS)
> >  virpcimock_la_LDFLAGS = $(MOCKLIBS_LDFLAGS)
> > -virpcimock_la_LIBADD = $(MOCKLIBS_LIBS)
> > +virpcimock_la_LIBADD = $(MOCKLIBS_LIBS) \
> > +	../src/libvirt_util.la
> 
> You forgot to include $(PROBES_O). But even with that added in, this will
> not fly. While you make it work under libvirt it fails without it because
> 'stat' is undefined but called. Look into virtestmock.c for explanation.
> 
> I'm not sure how to address this issue. I know about it for quite some time
> now but all my previous attempts to fix it (including this one) failed. I've
> tried to mimic what virtestmock.c does, but without any luck.

What is the actual build problem with the virpcimock ? It seems to work
fine for me right now.

FWIW, the original intenion was that the mock impls should not use any
libvirt APIs, just pure POSIX / Linux APIs, so as to avoid dependancies
which could end up being circular.

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://entangle-photo.org       -o-    http://search.cpan.org/~danberr/ :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux