-
Notifications
You must be signed in to change notification settings - Fork 64
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
Error while trying to get library: undefined: sync.Pool #40
Comments
sync.Pool was added in go 1.3. |
@pwaller Yeah, I know. But |
Do you have some hard constraint keeping you on Go 1.2.1? If not, I highly recommend updating to the latest 1.4.2. It will perform better and you will run into fewer bugs (a lot of issues have been fixed/improved since 1.2.1). Updating is easy and compatible with existing code. |
Ugh, I didn't know that I have to tag each minor version individually. Still, you're missing a ton of package development if you stick with 1.2.1, including bug fixes that will be undocumented to you. (I think we may have fixed a bug in the rotation matrices or something since then). Unfortunately, we just don't have the time/manpower to curate bugfixes for old Go versions. If you can, I'd really recommend at least upgrading to 1.2.2. If it's impossible, I'll try and add the 1.2.1 tag to the commit with the 1.2.2 tag, with the caveat that I really don't recommend using it. |
On review, perhaps we could refactor Mat/VecMN out of the core packages. If we switch to RMO in order to comply with the Go f32 package, we should probably get rid of them anyway, and encourage people to use github.com/gonum/matrix for MN functionality. |
I'm trying to get
github.com/go-gl/mathgl/mgl32
and getting the following error:Here is my version of go:
The text was updated successfully, but these errors were encountered: