Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

go1.11 compat broken #53

Open
markus-wa opened this issue Apr 25, 2022 · 2 comments
Open

go1.11 compat broken #53

markus-wa opened this issue Apr 25, 2022 · 2 comments

Comments

@markus-wa
Copy link
Contributor

markus-wa commented Apr 25, 2022

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
Copy link
Contributor Author

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
Copy link
Contributor Author

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 ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant