[libvirt] [PATCH] parthelper: fix compilation without optimization

Eric Blake eblake at redhat.com
Mon Jun 14 20:50:15 UTC 2010


Daniel's patch works with gcc and CFLAGS containing -O (the
autoconf default), but fails with non-gcc or with other
CFLAGS (such as -g), since c-ctype.h declares c_isdigit as
a macro only for certain compilation settings.

* src/Makefile.am (libvirt_parthelper_LDFLAGS): Add gnulib
library, for when c_isdigit is not a macro.
---

I don't see Daniel's patch applied upstream yet, but either
this should be squashed in, or applied immediately after.

However, given the state of:
https://bugzilla.redhat.com/show_bug.cgi?id=593785
I would recommend that this patch remain separate to
make it easier for backporting.

 src/Makefile.am |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 6bdf73c..dac12b9 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1037,7 +1037,7 @@ libexec_PROGRAMS += libvirt_parthelper

 libvirt_parthelper_SOURCES = $(STORAGE_HELPER_DISK_SOURCES)
 libvirt_parthelper_LDFLAGS = $(WARN_LDFLAGS) $(COVERAGE_LDFLAGS)
-libvirt_parthelper_LDADD = $(LIBPARTED_LIBS)
+libvirt_parthelper_LDADD = $(LIBPARTED_LIBS) ../gnulib/lib/libgnu.la
 libvirt_parthelper_CFLAGS =  $(LIBPARTED_CFLAGS)
 endif
 endif
-- 
1.7.0.1




More information about the libvir-list mailing list