Double restart? about fresh

[gravityblast]

@mitranim it's not common, but you can run it with the verbose option to see what's happening. is it possible that your editor saves twice and fresh compiles twice? you can try increasing the build_delay option

[mitranim]

I'm sorry but I don't see a verbose option in the configuration. Anyway what I'm seeing in the terminal is that watcher sends the MODIFY|ATTRIB event (for the same file) twice. This happens even when using the touch utility to modify the file, so this doesn't appear to be editor related.

[mitranim]

Some additional notes. The folder is a git repo; it's not inside Dropbox or anything like that, and I don't believe I have any daemons watching this location; quitting the editor and restarting Finder has no effect; the problem occurs regardless of which of the *.go files in the folder I touch. Would love to hear other troubleshooting suggestions. :)

[gravityblast]

I remember in the past it was an issue with fsnotify, that's why I introduced the delay option, to avoid this problem and just building once if I receive a second event just after the first one. it might be solved with go-fsnotify

[mitranim]

Increasing the delay to 2000 does solve the issue, but that's a lot of waiting. I'm not entirely sure what you mean by solving this with go-fsnotify. A clue would be appreciated.

[gravityblast]

yes, 2000 is too much, when I had that problem my editor was saving twice, but the second event was something like 100 ms after the first one.
about fsnotify, I'm using https://github.com/howeyc/fsnotify to watch the fs, but I think https://github.com/go-fsnotify/fsnotify is more up to date, switching to that may solve the issue, I'm not sure.

[mitranim]

Trying out your suggestion. Installed https://gopkg.in/fsnotify.v0 per the recommendation on https://github.com/go-fsnotify/fsnotify#api-stability, switched fresh to it, recompiled binary. No observable change so far. The watcher still fires two events per file modification. =( And the newer version of go-fsnotify has a different API. =/

[gmccue]

@mitranim is this still an issue?

[mitranim]

@gmccue Haven't checked since last year, and I don't remember how it ended up, sorry.