Fix for building against musl

Mike Frysinger vapier at gentoo.org
Tue Mar 17 22:49:31 UTC 2015


On 17 Mar 2015 15:18, Dmitry V. Levin wrote:
> On Tue, Mar 17, 2015 at 03:36:55AM -0400, Mike Frysinger wrote:
> > On 16 Mar 2015 21:42, Felix Janda wrote:
> > > <asm/ioctl.h> is for example needed for _IOC_SIZE(). That is
> > > likely also the reason why ioctl.c includes it.
> 
> evdev.c includes <linux/input.h> which includes <sys/ioctl.h>.
> On glibc, this leads to inclusion of <asm/ioctl.h>, musl may do things
> differently.
> 
> > we should stick to linux/ioctl.h imo, and update ioctl.c to match
> 
> I'm OK with including <asm/ioctl.h>; <linux/ioctl.h> includes other
> headers besides <asm/ioctl.h>.

i think generally we should be sticking to linux/xxx.h unless we specifically 
need something out of asm/xxx.h.  although i'm not sure what you mean in this 
case ...

$ cat include/uapi/linux/ioctl.h 
#ifndef _LINUX_IOCTL_H
#define _LINUX_IOCTL_H

#include <asm/ioctl.h>

#endif /* _LINUX_IOCTL_H */
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20150317/1bd6d5e5/attachment.bin>


More information about the Strace-devel mailing list