Re: [Xen-devel] [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h

2017-04-18 Thread Wei Liu 
On Tue, Apr 18, 2017 at 01:16:04PM +0300, Razvan Cojocaru wrote:
> On 04/18/2017 01:03 PM, Wei Liu wrote:
> > On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
> >> The POSIX spec specifies to use:
> >> #include 
> >> instead of:
> >> #include 
> >> as seen here:
> >>http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
> >>
> >> This removes the warning:
> >> #warning redirecting incorrect #include  to 
> >> when building with the musl C-library.
> >>
> >> Signed-off-by: Alistair Francis 
> > 
> > Acked-by: Wei Liu 
> 
> FWIW:
> 
> Acked-by: Razvan Cojocaru 
> 

Thanks!

> 
> Thanks,
> Razvan

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h

2017-04-18 Thread Razvan Cojocaru 
On 04/18/2017 01:03 PM, Wei Liu wrote:
> On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
>> The POSIX spec specifies to use:
>> #include 
>> instead of:
>> #include 
>> as seen here:
>>http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
>>
>> This removes the warning:
>> #warning redirecting incorrect #include  to 
>> when building with the musl C-library.
>>
>> Signed-off-by: Alistair Francis 
> 
> Acked-by: Wei Liu 

FWIW:

Acked-by: Razvan Cojocaru 


Thanks,
Razvan

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [Xen-devel] [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h

2017-04-18 Thread Wei Liu 
On Mon, Apr 17, 2017 at 02:33:11PM -0700, Alistair Francis wrote:
> The POSIX spec specifies to use:
> #include 
> instead of:
> #include 
> as seen here:
>http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html
> 
> This removes the warning:
> #warning redirecting incorrect #include  to 
> when building with the musl C-library.
> 
> Signed-off-by: Alistair Francis 

Acked-by: Wei Liu 

___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

[Xen-devel] [PATCH 2/2] tools: Use POSIX signal.h instead of sys/signal.h

2017-04-17 Thread Alistair Francis 
The POSIX spec specifies to use:
#include 
instead of:
#include 
as seen here:
   http://pubs.opengroup.org/onlinepubs/009695399/functions/signal.html

This removes the warning:
#warning redirecting incorrect #include  to 
when building with the musl C-library.

Signed-off-by: Alistair Francis 
---
 tools/blktap2/drivers/tapdisk-server.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/blktap2/drivers/tapdisk-server.c 
b/tools/blktap2/drivers/tapdisk-server.c
index eecde3d23f..71315bb069 100644
--- a/tools/blktap2/drivers/tapdisk-server.c
+++ b/tools/blktap2/drivers/tapdisk-server.c
@@ -30,7 +30,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 
 #include "tapdisk-utils.h"
 #include "tapdisk-server.h"
-- 
2.11.0


___
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel