From: Christoph Egger <chegger@amazon.de>
To: xen-devel <xen-devel@lists.xen.org>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>,
Ian Campbell <Ian.Campbell@citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: tools/libxl: fix compilation and link errors on NetBSD
Date: Tue, 14 May 2013 16:51:37 +0200 [thread overview]
Message-ID: <51924F79.4000204@amazon.de> (raw)
commit cad172d7b88bd443c81d865051297875ce2551bc
Author: Christoph Egger <chegger@amazon.de>
Date: Thu Feb 7 14:42:29 2013 +0000
tools/libxl: fix compilation and link errors on NetBSD
- Fix testidl link error that libyajl is not found
- Make linking of xl and testidl consistent
- fix error: array subscript has type 'char'
Signed-off-by: Christoph Egger <chegger@amazon.de>
Reviewed-by: Matthew Wilson <msw@amazon.de>
diff --git a/tools/Rules.mk b/tools/Rules.mk
index 3f03a31..4067955 100644
--- a/tools/Rules.mk
+++ b/tools/Rules.mk
@@ -56,7 +56,7 @@ SHLIB_libblktapctl =
endif
CFLAGS_libxenlight = -I$(XEN_XENLIGHT) $(CFLAGS_libxenctrl) $(CFLAGS_xeninclude)
-LDLIBS_libxenlight = $(XEN_XENLIGHT)/libxenlight.so $(SHLIB_libxenctrl) $(SHLIB_libxenstore) $(SHLIB_libblktapctl)
+LDLIBS_libxenlight = $(XEN_XENLIGHT)/libxenlight.so $(APPEND_LDFLAGS) -lyajl $(SHLIB_libxenctrl) $(SHLIB_libxenstore) $(SHLIB_libblktapctl)
SHLIB_libxenlight = -Wl,-rpath-link=$(XEN_XENLIGHT)
CFLAGS += -D__XEN_TOOLS__
diff --git a/tools/libxl/Makefile b/tools/libxl/Makefile
index cf214bb..c65c11e 100644
--- a/tools/libxl/Makefile
+++ b/tools/libxl/Makefile
@@ -189,7 +189,7 @@ libxlutil.a: $(LIBXLU_OBJS)
$(AR) rcs libxlutil.a $^
xl: $(XL_OBJS) libxlutil.so libxenlight.so
- $(CC) $(LDFLAGS) -o $@ $(XL_OBJS) libxlutil.so $(LDLIBS_libxenlight) $(LDLIBS_libxenctrl) -lyajl $(APPEND_LDFLAGS)
+ $(CC) $(LDFLAGS) -o $@ $(XL_OBJS) libxlutil.so $(LDLIBS_libxenlight) $(LDLIBS_libxenctrl) $(APPEND_LDFLAGS)
libxl-save-helper: $(SAVE_HELPER_OBJS) libxenlight.so
$(CC) $(LDFLAGS) -o $@ $(SAVE_HELPER_OBJS) $(LDLIBS_libxenctrl) $(LDLIBS_libxenguest) $(APPEND_LDFLAGS)
diff --git a/tools/libxl/libxl_utils.c b/tools/libxl/libxl_utils.c
index 35da71c..0c039b1 100644
--- a/tools/libxl/libxl_utils.c
+++ b/tools/libxl/libxl_utils.c
@@ -95,7 +95,7 @@ int libxl_domain_qualifier_to_domid(libxl_ctx *ctx, const char *name,
{
int i, rv;
for (i=0; name[i]; i++) {
- if (!isdigit(name[i])) {
+ if (!isdigit((unigned char)name[i])) {
goto nondigit_found;
}
}
next reply other threads:[~2013-05-14 14:51 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-14 14:51 Christoph Egger [this message]
2013-05-14 16:34 ` tools/libxl: fix compilation and link errors on NetBSD Ian Campbell
2013-05-15 10:31 ` Christoph Egger
2013-05-15 10:46 ` Ian Campbell
2013-05-16 8:48 ` Matt Wilson
2013-05-14 16:35 ` Keir Fraser
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51924F79.4000204@amazon.de \
--to=chegger@amazon.de \
--cc=Ian.Campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.