<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN"> <HTML> <HEAD> <META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1"> <META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7653.38"> <TITLE>RE: [libvirt] [PATCH 00/12] Domain Events - Round 2</TITLE> </HEAD> <BODY>  Great to hear this all works for you. I'll be sure to take that automake change (autoconf/automake is still a bit new to me, and I still find it a bit cryptic) As for the test app - I don't think that it is appropriate to run from make check It was more designed to be a developer example, as suggested here: <A HREF="https://www.redhat.com/archives/libvir-list/2008-October/msg00258.html">https://www.redhat.com/archives/libvir-list/2008-October/msg00258.html</A> -----Original Message----- From: Jim Meyering [<A HREF="mailto:jim@meyering.net">mailto:jim@meyering.net</A>] Sent: Sun 10/19/2008 6:23 AM To: Ben Guthro Cc: libvir-list@redhat.com Subject: Re: [libvirt] [PATCH 00/12] Domain Events - Round 2 Ben Guthro <bguthro@virtualiron.com> wrote: > The following patch series implements the Events API discussed here previously in this thread: > <A HREF="http://www.redhat.com/archives/libvir-list/2008-September/msg00321.html">http://www.redhat.com/archives/libvir-list/2008-September/msg00321.html</A> and > <A HREF="http://www.redhat.com/archives/libvir-list/2008-October/msg00245.html">http://www.redhat.com/archives/libvir-list/2008-October/msg00245.html</A> ... Nice work. I spotted no problems in review, and noticed only one when testing: In the new file, examples/domain-events/events-c/Makefile.am > diff --git a/examples/domain-events/events-c/Makefile.am b/examples/domain-events/events-c/Makefile.am ... > @@ -0,0 +1,5 @@ > +INCLUDES = -I@top_srcdir@/include > +bin_PROGRAMS = event-test > +event_test_CFLAGS = $(WARN_CFLAGS) > +event_test_SOURCES = event-test.c > +event_test_LDADD = -L@top_srcdir@/src/.libs -lvirt that would fail with a non-srcdir build (i.e., when the build_dir != srcdir), because then @top_srcdir@/src/.libs doesn't exist, and that causes "make distcheck" to fail. Also, it is best not to reference libtool's .libs/ directories directly, so you can use this instead: diff --git a/examples/domain-events/events-c/Makefile.am b/examples/domain-events/events-c/Makefile.am index 1788dac..3c63ca3 100644 --- a/examples/domain-events/events-c/Makefile.am +++ b/examples/domain-events/events-c/Makefile.am @@ -2,4 +2,4 @@ INCLUDES = -I@top_srcdir@/include bin_PROGRAMS = event-test event_test_CFLAGS = $(WARN_CFLAGS) event_test_SOURCES = event-test.c -event_test_LDADD = -L@top_srcdir@/src/.libs -lvirt +event_test_LDADD = @top_builddir@/src/libvirt.la Finally, I noticed that "make check" does not run the new event-test program that's built there. Should it? </BODY> </HTML>