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

dependencies #14

Closed
setop opened this issue Aug 12, 2023 · 4 comments
Closed

dependencies #14

setop opened this issue Aug 12, 2023 · 4 comments

Comments

@setop
Copy link

setop commented Aug 12, 2023

In order to make it work on Debian 11, I had to install many dependencies:

sudo apt-get install \
  libxcursor-dev \
  libxrandr-dev  \
  libxinerama-dev  \
  libgl-dev \
  libxi-dev \
  freeglut3-dev \
  libglu1-mesa-dev  \
  libglfw3-dev \
  libgles2-mesa-dev \
  libglu1-mesa-dev \
  mesa-common-dev \
  xorg-dev

go build

This may solve #9.

Plus I had to manually copy the font to the expected folder.

mkdir -p $HOME/go/src/github.com/esimov/diagram/font
cp font/gloriahallelujah.ttf $HOME/go/src/github.com/esimov/diagram/font/

Nice tool BTW !

@esimov
Copy link
Owner

esimov commented Aug 14, 2023

All of the above dependencies are related to the go-glfw library. That's why it's also mentioned in the README file, that glfw is required in order to run this tool.

@setop
Copy link
Author

setop commented Aug 14, 2023

All of the above dependencies are related to the go-glfw library. That's why it's also mentioned in the README file, that glfw is required in order to run this tool.

Yes, I've seen it.

They say:

On Ubuntu/Debian-like Linux distributions, you need libgl1-mesa-dev and xorg-dev packages.

Which is clearly not enough, at least on my Debian system.

On CentOS/Fedora-like Linux distributions, you need libX11-devel libXcursor-devel libXrandr-devel libXinerama-devel mesa-libGL-devel libXi-devel libXxf86vm-devel packages.

Combining with fedora-like entries help to get closer to the truth...

@esimov
Copy link
Owner

esimov commented Aug 16, 2023

You are welcome to create a PR and document the README file with the required dependencies.

@setop
Copy link
Author

setop commented Aug 22, 2023

It will not be as simple as a PR on the README.
It works on my laptop but to ensure it works everywhere, it would require some tests on a fresh install.
I already have a lot on my plate, I'll pass on this one and let this issue for reference.
Thanks for the discovery BTW.

@setop setop closed this as completed Aug 22, 2023
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

2 participants