[PATCH] parthelper: fix compilation without optimization

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

 



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

--
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]