* libxl: fix build failure for non-Linux platforms
@ 2011-05-26 9:33 Christoph Egger
2011-05-26 14:55 ` Ian Jackson
0 siblings, 1 reply; 2+ messages in thread
From: Christoph Egger @ 2011-05-26 9:33 UTC (permalink / raw)
To: xen-devel
[-- Attachment #1: Type: text/plain, Size: 446 bytes --]
Move variable definitions into Linux-specific sections where they are
actually used. Fixes warning about unused variables.
Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
--
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632
[-- Attachment #2: xen_tools_libxl.diff --]
[-- Type: text/plain, Size: 707 bytes --]
diff -r 87ff9d4de6f0 tools/libxl/libxl_exec.c
--- a/tools/libxl/libxl_exec.c Wed May 25 16:04:56 2011 +0200
+++ b/tools/libxl/libxl_exec.c Thu May 26 11:30:41 2011 +0200
@@ -41,8 +41,6 @@ static void check_open_fds(const char *w
const char *env_debug;
int debug;
int i, flags, badness = 0;
- char path[PATH_MAX];
- char link[PATH_MAX+1];
env_debug = getenv("_LIBXL_DEBUG_EXEC_FDS");
if (!env_debug) return;
@@ -53,6 +51,8 @@ static void check_open_fds(const char *w
for (i = 4; i < 256; i++) {
#ifdef __linux__
size_t len;
+ char path[PATH_MAX];
+ char link[PATH_MAX+1];
#endif
flags = fcntl(i, F_GETFD);
if ( flags == -1 ) {
[-- Attachment #3: Type: text/plain, Size: 138 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: libxl: fix build failure for non-Linux platforms
2011-05-26 9:33 libxl: fix build failure for non-Linux platforms Christoph Egger
@ 2011-05-26 14:55 ` Ian Jackson
0 siblings, 0 replies; 2+ messages in thread
From: Ian Jackson @ 2011-05-26 14:55 UTC (permalink / raw)
To: Christoph Egger; +Cc: xen-devel
Christoph Egger writes ("[Xen-devel] libxl: fix build failure for non-Linux platforms"):
> Move variable definitions into Linux-specific sections where they are
> actually used. Fixes warning about unused variables.
>
> Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>
Thanks.
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-05-26 14:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-26 9:33 libxl: fix build failure for non-Linux platforms Christoph Egger
2011-05-26 14:55 ` Ian Jackson
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.