Skip to content

kqueue: better error if watching a file fails#471

Merged
arp242 merged 1 commit intomainfrom
kqueue-err
Jul 29, 2022
Merged

kqueue: better error if watching a file fails#471
arp242 merged 1 commit intomainfrom
kqueue-err

Conversation

@arp242
Copy link
Copy Markdown
Member

@arp242 arp242 commented Jul 24, 2022

kqueue requires opening a file descriptor for every file; so we read the
directory and do that. If this failed the error was quite unclear; for
example:

"/tmp/fsnotify_permission_denied/": permission denied

This puts the filepath in there:

"/tmp/fsnotify_permission_denied/": "/tmp/fsnotify_permission_denied/test2.log": permission denied

Fixes #270

kqueue requires opening a file descriptor for every file; so we read the
directory and do that. If this failed the error was quite unclear; for
example:

	"/tmp/fsnotify_permission_denied/": permission denied

This puts the filepath in there:

	"/tmp/fsnotify_permission_denied/": "/tmp/fsnotify_permission_denied/test2.log": permission denied

Fixes #270
@arp242 arp242 merged commit cc15908 into main Jul 29, 2022
@arp242 arp242 deleted the kqueue-err branch July 29, 2022 18:28
@shogo82148 shogo82148 mentioned this pull request Mar 6, 2024
25 tasks
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.

Permission denied when try to watch for dir with one of files which have --wx------ permissions

1 participant