[libvirt] [PATCH] fix failing "make syntax-check"

"make syntax-check" just started failing. This fixes it: fix failing "make syntax-check" * src/stats_linux.c: Don't include <c-ctype.h> no longer used. --- ChangeLog | 3 +++ src/stats_linux.c | 1 - 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/ChangeLog b/ChangeLog index c23799c..ea6f1da 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,8 @@ Tue Aug 6 13:30:44 CEST 2008 Jim Meyering <meyering@redhat.com> + fix failing "make syntax-check" + * src/stats_linux.c: Don't include <c-ctype.h> no longer used. + work around MinGW build failure due to its lack of inet_aton Use inet_pton instead; pull in gnulib's module by the same name. * src/network_conf.c (virNetworkDefParseXML): Use inet_pton, diff --git a/src/stats_linux.c b/src/stats_linux.c index 897251b..3f7dd52 100644 --- a/src/stats_linux.c +++ b/src/stats_linux.c @@ -19,7 +19,6 @@ #include <string.h> #include <unistd.h> #include <regex.h> -#include "c-ctype.h" #ifdef WITH_XEN #include <xs.h> -- 1.6.0.rc1.92.g189f7

On Wed, Aug 06, 2008 at 02:26:52PM +0200, Jim Meyering wrote:
"make syntax-check" just started failing.
strange it works for me on CVS head
This fixes it:
fix failing "make syntax-check" * src/stats_linux.c: Don't include <c-ctype.h> no longer used.
+1 sure Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/

Daniel Veillard <veillard@redhat.com> wrote:
On Wed, Aug 06, 2008 at 02:26:52PM +0200, Jim Meyering wrote:
"make syntax-check" just started failing.
strange it works for me on CVS head
It was so small/obvious, that after waiting half an hour, I went ahead and committed it. If you add back the "#include <c-ctype.h>" line, "make syntax-check" should fail.
This fixes it:
fix failing "make syntax-check" * src/stats_linux.c: Don't include <c-ctype.h> no longer used.
+1 sure
Thanks for the review.
participants (2)
-
Daniel Veillard
-
Jim Meyering