All of lore.kernel.org
 help / color / mirror / Atom feed
* 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.