Revert "Merge pull request #1439 from ltouati/fsnotify"

[nkubala]

This reverts commit 3402a94, reversing
changes made to 3795677.

@ltouati unfortunately, this change requires cgo, which is incompatible with cross-compilation in Golang, effectively breaking our CI. This will need to be reworked to remove its dependency on cgo before it can go back in.

[dgageot]

There no way to remove the dependency on cgo. Let’s try to improve our release system for next release

[codecov-io]

Codecov Report

Merging #1508 into master will increase coverage by 0.27%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1508      +/-   ##
==========================================
+ Coverage      45%   45.27%   +0.27%     
==========================================
  Files         115      115              
  Lines        4802     4764      -38     
==========================================
- Hits         2161     2157       -4     
+ Misses       2420     2387      -33     
+ Partials      221      220       -1

Impacted Files	Coverage Δ
cmd/skaffold/app/cmd/dev.go	0% <0%> (ø)	⬆️
pkg/skaffold/watch/watch.go	86.04% <100%> (+3.43%)	⬆️
pkg/skaffold/watch/triggers.go	33.33% <66.66%> (+12.2%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8d452ca...03605a2. Read the comment docs.

[ltouati]

I'm happy to work on this as Skaffold is not usable with large projects if you don't use notifications. Can you please point me to the relevant part ?

Thx

[nkubala]

@ltouati thanks. unfortunately I think the responsibility lies with us, we need to come up with another CI solution that is able to build for all platforms because right now we rely on go cross compilation and this won't work with cgo. I'd like to see this go back in for the next release though, so we'll keep you posted.