Skip to content

Add traverse path filter for Linux#112

Open
gjz010 wants to merge 3 commits intohaskell-fswatch:masterfrom
gjz010-Forks:master
Open

Add traverse path filter for Linux#112
gjz010 wants to merge 3 commits intohaskell-fswatch:masterfrom
gjz010-Forks:master

Conversation

@gjz010
Copy link
Copy Markdown

@gjz010 gjz010 commented Mar 30, 2024

Currently the implementation of FSNotify on Linux traverses through all subdirectories and call INo.addWatch (inotify_add_watch) recursively on all directories. This crashes applications when a really unwanted directory (for example, one containing broken symlinks) is scanned, and may cause performance problems.

(See srid/unionmount#9 , where .direnv and therefore a whole nixpkgs is scanned.)

In this patch, a confPathFilter is added to WatchConfig to allow users to hold the library back from traversing down unwanted directories.

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.

1 participant