xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] libxl: add missing includes
@ 2012-02-09 13:14 Christoph Egger
  2012-02-20 18:44 ` Ian Jackson
  0 siblings, 1 reply; 4+ messages in thread
From: Christoph Egger @ 2012-02-09 13:14 UTC (permalink / raw)
  To: xen-devel@lists.xensource.com

[-- Attachment #1: Type: text/plain, Size: 443 bytes --]


include <poll.h> for struct pollfd
include <time.h> for struct timeval
Fixes gcc complaints about implicit declaration.

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: 354 bytes --]

diff -r f27d5c83b05e tools/libxl/libxl_event.h
--- a/tools/libxl/libxl_event.h	Mon Feb 06 14:12:36 2012 +0100
+++ b/tools/libxl/libxl_event.h	Thu Feb 09 14:10:33 2012 +0100
@@ -17,6 +17,8 @@
 #define LIBXL_EVENT_H
 
 #include <libxl.h>
+#include <poll.h>
+#include <time.h>
 
 /*======================================================================*/
 

[-- 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] 4+ messages in thread

* Re: [PATCH] libxl: add missing includes
  2012-02-09 13:14 [PATCH] libxl: add missing includes Christoph Egger
@ 2012-02-20 18:44 ` Ian Jackson
  2012-02-21  9:18   ` Christoph Egger
  0 siblings, 1 reply; 4+ messages in thread
From: Ian Jackson @ 2012-02-20 18:44 UTC (permalink / raw)
  To: Christoph Egger; +Cc: xen-devel@lists.xensource.com

Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):
> 
> include <poll.h> for struct pollfd
> include <time.h> for struct timeval
> Fixes gcc complaints about implicit declaration.

According to SuS, it should be <sys/time.h>.

Ian.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] libxl: add missing includes
  2012-02-20 18:44 ` Ian Jackson
@ 2012-02-21  9:18   ` Christoph Egger
  2012-02-21 16:44     ` Ian Jackson
  0 siblings, 1 reply; 4+ messages in thread
From: Christoph Egger @ 2012-02-21  9:18 UTC (permalink / raw)
  To: Ian Jackson; +Cc: xen-devel@lists.xensource.com

On 02/20/12 19:44, Ian Jackson wrote:
> Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):
>>
>> include<poll.h>  for struct pollfd
>> include<time.h>  for struct timeval
>> Fixes gcc complaints about implicit declaration.
>
> According to SuS, it should be<sys/time.h>.

Yes, sorry. Should I resend?

Christoph


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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] libxl: add missing includes
  2012-02-21  9:18   ` Christoph Egger
@ 2012-02-21 16:44     ` Ian Jackson
  0 siblings, 0 replies; 4+ messages in thread
From: Ian Jackson @ 2012-02-21 16:44 UTC (permalink / raw)
  To: Christoph Egger; +Cc: xen-devel@lists.xensource.com

Christoph Egger writes ("Re: [Xen-devel] [PATCH] libxl: add missing includes"):
> On 02/20/12 19:44, Ian Jackson wrote:
> > Christoph Egger writes ("[Xen-devel] [PATCH] libxl: add missing includes"):
> >>
> >> include<poll.h>  for struct pollfd
> >> include<time.h>  for struct timeval
> >> Fixes gcc complaints about implicit declaration.
> >
> > According to SuS, it should be<sys/time.h>.
> 
> Yes, sorry. Should I resend?

I have fixed it up and committed it.

Thanks,
Ian.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-02-21 16:44 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-02-09 13:14 [PATCH] libxl: add missing includes Christoph Egger
2012-02-20 18:44 ` Ian Jackson
2012-02-21  9:18   ` Christoph Egger
2012-02-21 16:44     ` Ian Jackson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).