Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implementation of events which require kernel patching #32

Merged
merged 13 commits into from Aug 29, 2015
Merged

implementation of events which require kernel patching #32

merged 13 commits into from Aug 29, 2015

Conversation

wulf7
Copy link
Contributor

@wulf7 wulf7 commented Jul 12, 2015

Implement IN_OPEN, IN_CLOSE and IN_ACCESS inotify events
Implement inotify_init1
Bring errno into line with documentation

wulf7 added 13 commits July 9, 2015 16:10
…file

It becomes too highly integrated with watch internals.
It`s possible to use stat(2) S_* macroses on watch flags now.
It is done via adding new kqueue events: NOTE_OPEN, NOTE_CLOSE and NOTE_READ
FreeBSD kernel patch: patches/freebsd11-NOTE_OPEN-NOTE_CLOSE-NOTE_READ.patch
Do a dup/rewind instead of reopening directory on listing when it possible
This allows to separate IN_MOVE inotify events from IN_CREATE and IN_DELETE ones
when file was moved in or out of watched directory.
While here, align some misaligned define values and comments
While here, improve pipe closing code path
dmatveev added a commit that referenced this pull request Aug 29, 2015
implementation of events which require kernel patching
@dmatveev dmatveev merged commit b2873cf into dmatveev:master Aug 29, 2015
@dmatveev
Copy link
Owner

Tested on my side and merged. Still got "Signal 13" during one of the test runs BTW.
And, again, shame on me for doing it a month late :(

@wulf7
Copy link
Contributor Author

wulf7 commented Sep 15, 2015

Still got "Signal 13" during one of the test runs BTW

It seems to be a Net/OpenBSD kernel bug. I`ll try to workaround it when got some spare time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants