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

RE: [libvirt] [PATCH 00/12] Domain Events - Round 2



Title: RE: [libvirt] [PATCH 00/12] Domain Events - Round 2

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:
https://www.redhat.com/archives/libvir-list/2008-October/msg00258.html


-----Original Message-----
From: Jim Meyering [mailto:jim meyering net]
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:
> http://www.redhat.com/archives/libvir-list/2008-September/msg00321.html and
> http://www.redhat.com/archives/libvir-list/2008-October/msg00245.html
...

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?


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