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

Update README.md to fix installation error #688

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

centerorbit
Copy link

Added a note about needing to install libgles-dev package if you get an error during install.

@sbinet
Copy link
Member

sbinet commented May 3, 2021

thanks. (it's probably missing a few other Gio dependencies, though)

perhaps we should "guard" vg/vggio behind a // +build vggio build tag, so go get gonum.org/v1/plot/... is kept "pure-Go" ?
@kortschak

@kortschak
Copy link
Member

Yeah, I'm not sure. That places an obstacle to use of the package that causes its own problems.

We should also decide whether we want to test on travis, since we have minutes back. I'd prefer not to since we're half way through what we have and they are more valuable for testing gonum/gonum on arm64.

@centerorbit
Copy link
Author

centerorbit commented May 3, 2021

Thanks all. Just for reference, I experienced this on Pop!_OS 20.10; and it was the only error I experienced during install.


Note: If you get this error during install "Package glesv2 was not found in the pkg-config search path.", install the `libgles-dev` package for your system:

`sudo apt install libgles-dev`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps redirect (also? instead?) to Gio's official page for installation?
we should probably rely on their instructions than ours as the latter might sync' off...

ie: https://gioui.org/doc/install

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

Successfully merging this pull request may close these issues.

3 participants