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
Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go 1.6
What operating system and processor architecture are you using (go env)?
amd64, Linux
What did you do?
in inotify_linux RemoveWatch does not remove from the registry of watches if Linux returns an error.
I notify returns an error if you need to stop watching a directory, as it should because it automatically removes directories, but you still need to remove the registry because if the directory gets added back you need a new watcher reference id.
What did you expect to see?
when a directory is removed and added back files within the directory should return the full path name
What did you see instead?
If a directory is removed and added just the file name with no path is returned for the Event.Name
The text was updated successfully, but these errors were encountered:
bradfitz
changed the title
Inotify_linux RemoveWatch needs to delete reference to watch even if system returns error.
x/exp/inotify: inotify_linux RemoveWatch needs to delete reference to watch even if system returns error.
Mar 7, 2016
Please answer these questions before submitting your issue. Thanks!
go version
)?go 1.6
go env
)?amd64, Linux
in inotify_linux RemoveWatch does not remove from the registry of watches if Linux returns an error.
I notify returns an error if you need to stop watching a directory, as it should because it automatically removes directories, but you still need to remove the registry because if the directory gets added back you need a new watcher reference id.
when a directory is removed and added back files within the directory should return the full path name
If a directory is removed and added just the file name with no path is returned for the Event.Name
The delete function just needs to be moved
Fix can be found here:
jasonrichardsmith/exp@6d6db11
The text was updated successfully, but these errors were encountered: