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

Modified inotify update condition #208

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

obi1
Copy link
Contributor

@obi1 obi1 commented Dec 28, 2019

If using nfs or samba files get closed with CLOSE_NOWRITE or their
attributes get changed (ATTRIB). If inotify doesn't report these events
a lot of files gets missed due to not being readable.

obi1 added 4 commits November 30, 2019 21:31
If a browser uses temporary files in the same folder as destination
a inotify event occurs for the temporary file and is added by libtorrent
before the rename happens. In this case when removing the torrent the
file associated with it did not exist anymore. Sometimes the rename
happened just before libtorrent wanted to add the temporary file. In
that case the torrent was not added to downloads. This fix tells
libtorrent to just add files ending in .torrent. Maybe a better fix
would be to add a rename handler.
If using nfs or samba files get closed with CLOSE_NOWRITE or their
attributes get changed (ATTRIB). If inotify doesn't report these events
a lot of files gets missed due to not being readable.
If using nfs or samba files get closed with CLOSE_NOWRITE or their
attributes get changed (ATTRIB). If inotify doesn't report these events
a lot of files gets missed due to not being readable.
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