ifacemaker icon indicating copy to clipboard operation
ifacemaker copied to clipboard

go1.11 compat broken

Open markus-wa opened this issue 2 years ago • 2 comments

hi

this is just to inform that master now broke go1.11 compat. not sure if this is just a documentation issue, but v1.1.0 was working fine.

I noticed this via my CI pipeline breaking: https://github.com/markus-wa/demoinfocs-golang/runs/6155103076?check_suite_focus=true

to repro, use docker run -it golang:1.11

$ go get github.com/vburenin/ifacemaker
# golang.org/x/mod/module
src/golang.org/x/mod/module/module.go:147:5: undefined: errors.As

unfortunately go1.11 doesn't allow for specifying a specific version in GOPATH mode, so GO111MODULE=off go get github.com/vburenin/[email protected] doesn't fix the issue

$ go install github.com/vburenin/[email protected]
can't load package: package github.com/vburenin/[email protected]: cannot use path@version syntax in GOPATH mode

some options to fix:

  • A: explicitly require specific minimal version of Go in README
  • B: (not sure if this will even work) change module name to */v2 (may need to revert master to v1.1.0 first, then do that and re-apply the changes since then. not sure how go handles it)
  • C: restore go1.11 compat

Probably not worth doing option B as Go 1.11 is very old, so I'd go with option A or C.

markus-wa avatar Apr 25 '22 13:04 markus-wa

actually this is a bit of a bigger issue than I thought

option B doesn't seem to work option A means to use the package with old versions the user has to manually git clone the repo, check out a good version and go install

markus-wa avatar Apr 25 '22 14:04 markus-wa

further, v1.1.0 is a bit outdated. a new tag of the latest version with go1.11 compat (e.g. from maybe 2020 or early 2022?) would be very useful.

and PS: thanks a lot for maintaining the project :heart:

markus-wa avatar Apr 25 '22 14:04 markus-wa