search

rjeczalik / notify

File system event notification library on steroids.
MIT License
906 stars 128 forks source link

Switch to InotifyInit1 syscall for mips64/etc #122

Closed claytono closed 7 years ago

claytono commented 7 years ago

The InotifyInit syscall doesn't exist on all architectures, but InotifyInit1 does. In the kernel implementation, inotify_init is just a wrapper around inotify_init1 and passes in zero for the only parameter which is a flags field.

ppknap commented 7 years ago

Thanks @claytononeill !

Could you also add unix.IN_CLOEXEC as an argument to InotifyInit1 ? I was planning to switch this call but the idea must have slipped my mind 😶

claytono commented 7 years ago

@ppknap Updated with the IN_CLOEXEC flag.

ppknap commented 7 years ago

Thanks @claytononeill !