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

Add inotify support on BSD #273

Closed
chrisduerr opened this issue Dec 16, 2020 · 2 comments
Closed

Add inotify support on BSD #273

chrisduerr opened this issue Dec 16, 2020 · 2 comments
Labels
A-enhancement Z-needs implementation Needs an implementation, will accept PRs

Comments

@chrisduerr
Copy link

So a while back someone reported the following issue to Alacritty: alacritty/alacritty#3871

This isn't much of a surprise, it looks like notify's BSD implementation is just using the worst, most general fallback which causes high system load due to a relatively high poll rate set by Alacritty.

But it seems like at least some BSDs do have the ability to support inotify and might potentially have other native APIs too (I'm not familiar with those at least). I'm curious if this has been investigated, I'd assume that BSD just has not been a priority?

@deepy
Copy link

deepy commented Dec 17, 2020

For BSD support you'd probably want kqueue, which has the benefit of also being supported on macOS.
Judging by #136 and #117 there seems to be some interest but mostly everything seems abadoned

@xanderio
Copy link
Contributor

xanderio commented Jul 9, 2021

With #335 merged, i don't think this is necessary any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-enhancement Z-needs implementation Needs an implementation, will accept PRs
Projects
None yet
Development

No branches or pull requests

5 participants