CompileDaemon icon indicating copy to clipboard operation
CompileDaemon copied to clipboard

Bump fsnotify to bump sys for go1.18

Open ypresto opened this issue 3 years ago • 5 comments

I'm using go 1.18 on macOS, and go install fails with below message:

$ go install github.com/githubnemo/CompileDaemon@latest
# golang.org/x/sys/unix
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/syscall_darwin.1_13.go:25:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.1_13.go:27:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.1_13.go:40:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:28:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:43:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:59:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:75:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:90:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:105:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:121:3: //go:linkname must refer to declared function or variable
../../go/1.18.0/pkg/mod/golang.org/x/[email protected]/unix/zsyscall_darwin_amd64.go:121:3: too many errors

This is caused by old x/sys library.

Using depth dependency tree visualizer, I found old fsnotify module references old x/sys module.

So just I bumped it :)

ypresto avatar Apr 06 '22 08:04 ypresto

https://github.com/bwplotka/bingo test suite on MacOS / go1.18 does not work anymore because of this.

We would appreciate if this could be merged and released promptly :)

Thanks.

Needed for https://github.com/bwplotka/bingo/pull/109

sylr avatar Apr 07 '22 09:04 sylr

@githubnemo can you please merge this?

jeffreydwalter avatar Jun 02 '22 02:06 jeffreydwalter

I created fork at https://github.com/ypresto/CompileDaemon/tree/bump-sys . You can installed it by below snippet:

go get github.com/ypresto/CompileDaemon@bump-sys-fork

ypresto avatar Jun 27 '22 09:06 ypresto

Bump.

nhatthm avatar Sep 08 '22 15:09 nhatthm

@githubnemo Any chance this can get merged soon?

patrickvalle avatar Nov 27 '22 17:11 patrickvalle