You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Had to catch issues in two places to at least make it tolerate inaccessible directories.
__load_trees: catch a PermissionError from os.listdir
add_watch: check with os.access before inotify_add_watch
And that's just that one issue with permissions. There is more. If I had the time, I'd do my own library. This here just doesn't stand up to real-world usage. There are deeper issues here that require me to just wrap it in a loop and sleep if failed construct.
Just posting this to let others know. In contrast to other inotify wrappers for Python, this one isn't "done" (no need to further develop). It's got bugs, and it's abandoned.
The text was updated successfully, but these errors were encountered:
As the title says.
Had to catch issues in two places to at least make it tolerate inaccessible directories.
__load_trees
: catch aPermissionError
fromos.listdir
add_watch
: check withos.access
beforeinotify_add_watch
And that's just that one issue with permissions. There is more. If I had the time, I'd do my own library. This here just doesn't stand up to real-world usage. There are deeper issues here that require me to just wrap it in a loop and sleep if failed construct.
Just posting this to let others know. In contrast to other inotify wrappers for Python, this one isn't "done" (no need to further develop). It's got bugs, and it's abandoned.
The text was updated successfully, but these errors were encountered: