rpodgorny / unionfs-fuse

union filesystem using fuse
Other
300 stars 76 forks source link

Fixes ioctl.h include on OSX (and perhaps others). #50

Closed fhgwright closed 8 years ago

fhgwright commented 8 years ago

Including asm/ioctl.h on OSX doesn't work, but sys/ioctl.h (as specified in the man page) does. Man pages for Linux, FreeBSD, OpenBSD, and NetBSD all also specfy sys/ioctl.h, so it's unclear where asm/ioctl.h came from.

It appears that this include was originally added as asm/ioctl.h, so it was never changed from sys/ to asm/ to fix anything. Instead, it's probably just been incorrect all along, but happened to work where tested.

TESTED: Now builds (and works) on OSX. Also builds on Ubuntu.

rpodgorny commented 8 years ago

merged. thanks a lot!