xen-devel

[Top] [All Lists]

Re: [Xen-devel] [PATCH] libxl: build fix

from [Christoph Egger]

[Permanent Link][Original]

To:	Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject:	Re: [Xen-devel] [PATCH] libxl: build fix
From:	Christoph Egger <Christoph.Egger@xxxxxxx>
Date:	Tue, 13 Apr 2010 13:49:44 +0200
Cc:	xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date:	Tue, 13 Apr 2010 04:50:58 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to:	<19396.21435.748085.75231@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help:	<mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id:	Xen developer discussion <xen-devel.lists.xensource.com>
List-post:	<mailto:xen-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References:	<201004131216.16083.Christoph.Egger@xxxxxxx> <19396.21435.748085.75231@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent:	KMail/1.9.10

On Tuesday 13 April 2010 13:21:31 Ian Jackson wrote:
> Christoph Egger writes ("[Xen-devel] [PATCH] libxl: build fix"):
> > Attached patch makes libxl compile on NetBSD.
> > <sys/signal.h> is needed to get definition for SIGPIPE and SIG_IGN.
>
> <sys/signal.h> is not a standard header file.  It should be <signal.h>.
>
> Adding
>  #include <signal.h>
> would be correct.

On NetBSD, the SIG* defines are in <sys/signal.h> which is just
included by <signal.h>
On NetBSD, both works.

But you're right, the portable fix is the prefered one.

Christoph


-- 
---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen
Geschaeftsfuehrer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-devel] [PATCH] libxl: build fix, Christoph Egger Re: [Xen-devel] [PATCH] libxl: build fix, Ian Jackson Re: [Xen-devel] [PATCH] libxl: build fix, Christoph Egger <= Re: [Xen-devel] [PATCH] libxl: build fix, Ian Jackson Re: [Xen-devel] [PATCH] libxl: build fix, Christoph Egger

Previous by Date:	Re: [Xen-devel] [PATCH] libxl: build fix, Ian Jackson
Next by Date:	Re: [Xen-devel] radeon in dom0/ivtv in domU: irq 16 nobody cared, Konrad Rzeszutek Wilk
Previous by Thread:	Re: [Xen-devel] [PATCH] libxl: build fix, Ian Jackson
Next by Thread:	Re: [Xen-devel] [PATCH] libxl: build fix, Ian Jackson
Indexes:	[Date] [Thread] [Top] [All Lists]