rjeczalik / notify

File system event notification library on steroids.
MIT License
902 stars 128 forks source link

Panic in newWatcher on old FreeBSD #149

Closed calmh closed 6 years ago

calmh commented 6 years ago

The following panic happens on FreeBSD 8.3. The trace is from a vendored fork, but applies here as well. I'm not expecting things to work on FreeBSD 8.3 (the "not implemented" is surely because features are missing kernel-side), just to not panic and return an error that can be handled by the library caller instead.

panic: function not implemented

goroutine 1 [running]:
github.com/syncthing/syncthing/vendor/github.com/zillode/notify.newWatcher(0xc42001e300, 0x80, 0xc42001e300)
        /tmp/syncthing-e02d0de3/src/github.com/syncthing/syncthing/vendor/github.com/zillode/notify/watcher_trigger.go:109 +0x1e8

github.com/syncthing/syncthing/vendor/github.com/zillode/notify.newTree(0xb5029d, 0x17)
        /tmp/syncthing-e02d0de3/src/github.com/syncthing/syncthing/vendor/github.com/zillode/notify/tree.go:17 +0x4d

(panic trace truncated because user report... but still useful enough I think.)