revise contributing

update readme

Test on Go 1.18 and two most recent versions (#411)

* Test on Go 1.18 and two most recent versions

* on push

* ci

Update issue templates (#410)

* Update issue templates

* remove old issue template

Removed dead link

prepare 1.5.1, retract 1.5.0

Revert "Add AddRaw to not follow symlinks + Fix link folloing on Windows (#289)"

This reverts commit e2e95171bc7c7402864e613c8b00a768fbcc2380.

revise pull request template

v1.5.0 preparation (#380)

Add AddRaw to not follow symlinks + Fix link folloing on Windows (#289)

Update test matrix for go 1.17 stable release (#385)

add //go:build lines + add 1.17.0-rc2 to test matrix (#377)

Update x/sys to latest (#379)

Drop support/testing for Go 1.11 and earlier (#381)

Remove Travis CI and references

Add lint+vet+old versions to GitHub Action

introduce GitHub Actions

Fix unsafe pointer conversion (#325)

Fix the following unsafe pointer conversion found using the Go 1.14
-d=checkptr gcflags (`go1.14rc1 test -gcflags=all=-d=checkptr -v ./...`)

```
fatal error: checkptr: unsafe pointer conversion

goroutine 68 [running]:
runtime.throw(0x5833e9, 0x23)
runtime/panic.go:1112 +0x72 fp=0xc00030fba8 sp=0xc00030fb78 pc=0x432a32
runtime.checkptrAlignment(0xc00031f018, 0x544800, 0x1)
runtime/checkptr.go:18 +0xb7 fp=0xc00030fbd8 sp=0xc00030fba8 pc=0x4063a7
github.com/fsnotify/fsnotify.(*Watcher).readEvents(0xc0002900f0)
github.com/fsnotify/fsnotify/inotify.go:275 +0x457 fp=0xc00031ffd8 sp=0xc00030fbd8 pc=0x519037
runtime.goexit()
runtime/asm_amd64.s:1375 +0x1 fp=0xc00031ffe0 sp=0xc00031ffd8 pc=0x463eb1
created by github.com/fsnotify/fsnotify.NewWatcher
github.com/fsnotify/fsnotify/inotify.go:59 +0x1a5
```

Fixes #330

Move example usage to the readme

may resolve #328

Updating the FAQ section for supportability with NFS & FUSE filesystems.

closes #312 #306

drop 1.8.x from Travis CI

see #310

2019

revise editor config

tabs only for Go

Add Go modules

readme: plan to support fanotify

Open files with CLOEXEC

Fix #272

Also create epoll and pipe fds with close-on-exec (#155) (#219)

* Add the unix.O_CLOEXEC to the Pipe2 call
* Add unix.EPOLL_CLOEXEC to the Epoll call

readme: remove AppVeyor badge

Use Travis CI for Windows #284

Add windows to travis matrix

Fix lint issue with error doc string

only run golint on latest version of go

Check if channels are closed in the example (#244)

* Check if channels are closed in the example

* Check if the channels are closed before printing

darwin tests: Exchangedata is deprecated on 10.13

#264

travis: less verbose tests

TravisCI: skip tip

[#234] t.Fatalf -> t.Errorf in go routines (#266)

The documentation for t.FailNow (which is called by t.Fatalf) states
that it should not be called in go routines outside of the test go
routine.

Fix TestInotifyOverflow (#265)

* Queued inotify events could have been read by the test before max_queued_events was hit

TravisCI: updating test versions

v1.4.7

closes #222

[ci skip]

travis: test on Go 1.9.x

Fix missing verb on format string (#228)

Fix possible deadlock on closing the watcher on kqueue (#230)

* avoid deadlocks on Close()

raw channel work (not inside a select) should
always be prohibited in production code, as
it readily causes deadlocks on shutdown.

Also adds the test TestWatcherClose from #145.
This request duplicates that test, with two
lines fixed to address the houndcli-bot review
concerns.

Fixes #187
Fixes #145

* cleanup and simpler test

* also fix #225

* fix tests

Fix deadlock in Remove (linux/inotify)

Several people have reported this issue where if you are using a
single goroutine to watch for fs events and you call Remove in
that goroutine it can deadlock. The cause for this is that the Remove
was made synchronous by PR #73. The reason for this was to try and
ensure that maps were no longer leaking.

In this PR: IN_IGNORE was used as the event to ensure map cleanup.
This worked fine when Remove() was called and the next event was
IN_IGNORE, but when a different event was received the main goroutine
that's supposed to be reading from the Events channel would be stuck
waiting for the sync.Cond, which would never be hit because the select
would then block waiting for someone to receive the non-IN_IGNORE event
from the channel so it could proceed to process the IN_IGNORE event that
was waiting in the queue. Deadlock :)

Removing the synchronization then created two nasty races where Remove
followed by Remove would error unnecessarily, and one where Remove
followed by an Add could result in the maps being cleaned up AFTER the
Add call which means the inotify watch is active, but our maps don't
have the values anymore. It then becomes impossible to delete the
watches via the fsnotify code since it checks it's local data before
calling InotifyRemove.

This code attempts to use IN_DELETE_SELF as a means to know when a watch
was deleted as part of an unlink(). That means that we didn't delete the
watch via the fsnotify lib and we should clean up our maps since that
watch no longer exists. This allows us to clean up the maps immediately
when calling Remove since we no longer try to synchronize cleanup
using IN_IGNORE as the sync point.

- Fix #195
- Fix #123
- Fix #115

Watch.Add improvements (avoid race, fix consistency, reduce garbage) (#189)

* hold mutex for longer to avoid race condition

If Watcher.Add is called with the same name from multiple goroutines and
name is not already in the watches map then the last call to complete
wins and the watch created by the first call to complete is lost.

This commit holds the mutex for the full duration of the update to the
watches map.

* avoid clobbering existing flags if inotify_add_watch fails

Previously, the watch entry is the watches map was updated before the
call to InotifyAddWatch. If InotifyAddWatch fails then this left the
watch entry in an inconsistent state.

* reuse existing watch entry when possible

travis: update for Go 1.8

two most recent stable versions + tip

Moved FAQ into the README

closes #186
ref #129

[ci skip]

Properly handle inotify's IN_Q_OVERFLOW event (#149)

* Properly handle inotify's IN_Q_OVERFLOW event

Upon receiving an event with IN_Q_OVERFLOW set in the mask, generate an
error on the Errors chan, so that the application can take appropriate
action.

* Use a well-defined error (ErrEventOverflow) for inotify overflow

* Add a test for inotify queue overflow

replace references to OS X with macOS

[ci skip]

v1.4.2

pull request and issue template (#179)

* editor config sets 4 spaces per tab

via https://github.com/isaacs/github/issues/170#issuecomment-181814560

[ci skip]

* pull request and issue templates

https://help.github.com/articles/helping-people-contribute-to-your-project/

[ci skip]

* issue template: provide commands to get OS version

suggested by @mdlayher

[ci skip]

Create inotify fd with close-on-exec (#178)

* Create inotify fd with close-on-exec

* switch to unix.IN_CLOEXEC

v1.4.1

deflake inotify stress test (#177)

* deflake inotify stress test

* tab -> space

* add sleep between creation / deletion to ensure creation events are triggered.

rm go1.5.4 from travis config since the linter doesn't build in 1.5 (#175)

readme: remove gocover.io badge

[ci skip]

update link to contributing to external git repos

[ci skip]

v1.4.0

add @oozie to AUTHORS

[ci skip]

Add a String() func to Event.Op (#165)

* Add a String() method to Event.Op

closes #84

* Covering all 5 Op types by the test

Bump Travis to Go 1.6.3 (#156)

v1.3.1

Use filepath.Join() (#153)

fix #151

global: switch to x/sys/unix

The syscall package is locked since 1.4[1], all new changes are in x/sys[2]
The fixes for epoll/arm64 are going to be loaded to x/sys/unix. This
commit is straightforward search/replace with a couple of hand edits.

This is needed for fixing #130

[1] https://golang.org/s/go1.4-syscall
[2] https://godoc.org/golang.org/x/sys/unix

closes #135

Update Travis config to Go 1.5.4 and 1.6.1

Get fsnotify to build clean on Solaris

Signed-off-by: Amit Krishnan <krish.amit@gmail.com>
closes #128

Add more Go versions to Travis config

Append missing email address in AUTHORS

add Bjørn to authors

update authors

fix spelling error

thanks to Go Report Card

https://goreportcard.com/report/github.com/fsnotify/fsnotify

Fix golint errors in windows.go for #116

closes #121
fixes #116

rename organization to fsnotify

ref #108

kqueue: fix Remove-logic

`os.Lstat` does not return `error` when a file/directory exists,
so the code path  to check for newly created files after a REMOVE
on BSD was never executed.

Also, if the watched entry is a file, it has been removed from the watcher, so do a new-file-check for them as well.

This commit fixes issues on TextMate on OS X and others that uses `exchangedata` to do atomic saves.

See #17. closes #111.

v1.2.8

Add race detection on Travis CI.

thanks to @ppknap for the suggestion

kqueue: fix race condition in Close

fixes #99
closes #105

thanks to @djui for reporting the issue and @ppknap for writing a test.

Add test for concurrent file removes and watcher.Close calls

inotify: fix race in test

kqueue: avoid infinite loops from symlinks cycles

The semenatics of using kqueue are already slightly different than
inotify, specifically that inotify will give you events for the symlink
itself whereas kqueue will only give events for the target of the
symlink. This preserves that behaviour, that symlinks don't get notified
on, but ensures that circular symlinks don't recurse forever.

This change pushes the resolved filename up through the stack so that
the fileExists map can be properly populated; not doing so would mean
that deleting a cyclical symlink and then replacing it with a file of
the same name would not notify for that file's creation (or subsequent
events) because we would believe that the file already existed.

v1.2.4

kqueue: watch for rename events on subdirectories

if flags has no  syscall.NOTE_RENAME, subdir rename event will not notify.

closes #83

inotify: sync Watcher with underlying inotify with IN_IGNORE

Watcher.watches and paths are not deleted when entry was removed automatically. Deleting those entry at ignoreLinux() to fix it.

This changes removing invalid watch name, it (should) always return "can't remove non-existent..."

closes #73

inotify: switch to epoll_create1

Epoll_create was replaced with epoll_create1 in linux 2.6.27. The older
syscall isn't available on new architectures (such as arm64).

closes #100

v1.2.1

documentation before coverage

[skip ci]

remove CircleCI for OS X tests

TravisCI handles it

TravisCI update

Don't watch named pipes

You can't watch a named pipe, same as a socket. The call to `Open` will just hang forever.

closes #98

add related projects to Readme

by @rjeczalik and @samjacobson

v1.2.0

closes #66 (apply mail)

When removing, delete from watches before removing from inotify (fixes go-fsnotify/fsnotify#40)

Add stressy test to inotify.

Make wakeup pipe non-blocking

Reduce repeated error code in inotify

Minor doc and test typos

Fix fdPoller.close to only close when fds have been assigned

Properly clean fds when newFdPoller fails

Clean up closing file descriptors in newFdPoller

Renamed two misnamed things

Add some unit tests for inotify_poller

Close inotify fd from watcher, not from poller

Comment about the buffer size to epoll_wait