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

glew: add option for glu #24033

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

Conversation

xyz1001
Copy link
Contributor

@xyz1001 xyz1001 commented May 20, 2024

Specify library name and version: glew/2.2.0

glew with mesa-glu cause crash on my device. glu/system works well.
Environment:
OS: Kubuntu 22.04
Hardware: Lenovo ThinkBook 14p G2 ACH with AMD 5800H


Copy link
Contributor

🤖 Beep Boop! This pull request is making changes to 'recipes/glew//'.

👋 @Hopobcn you might be interested. 😉

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 2 (6d2ae7640efb5fcea48eb5ad859b4fcf2ff030cb):

  • glew/2.2.0:
    All packages built successfully! (All logs)

  • glew/2.1.0:
    All packages built successfully! (All logs)


Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 2 (6d2ae7640efb5fcea48eb5ad859b4fcf2ff030cb):

  • glew/2.1.0:
    All packages built successfully! (All logs)

  • glew/2.2.0:
    All packages built successfully! (All logs)

@valgur
Copy link
Contributor

valgur commented May 20, 2024

@jwillikers

@jwillikers
Copy link
Contributor

jwillikers commented May 20, 2024

@xyz1001 Thanks for submitting the PR regarding this issue! I'm curious as to why it would crash at runtime. Are you running your application with the conanrun.sh shell script? That's important as it will put the runtime-loaded libGLU from the standard Conan package in LD_LIBRARY_PATH path.

@xyz1001
Copy link
Contributor Author

xyz1001 commented May 21, 2024

@jwillikers
I am not running my application with conanrun.sh. I collect all so file from dependencies and copy them to the directory of executable file.
image

And then I run my application with LD_LIBRARY_PATH set to the directory of executable file.
image

It crash when call glDrawArrays.
image

@jwillikers
Copy link
Contributor

jwillikers commented May 21, 2024

@xyz1001 Thanks! That helps me understand the situation. I'm not certain GLU is the issue yet. What happens if you delete libglvnd from the .so files you've copied and try re-running?

@xyz1001
Copy link
Contributor Author

xyz1001 commented May 22, 2024

@jwillikers It works well after I delete all so copy from libglvnd. It turn to load those so from system packages.

@jwillikers
Copy link
Contributor

jwillikers commented May 22, 2024

@jwillikers It works well after I delete all so copy from libglvnd. It turn to load those so from system packages.

@xyz1001 Thanks for checking. I think making opengl/system an alternative for libglvnd in mesa/glu would solve your problem. I'm not sure the issue is related to the libglvnd Conan package or just the fact that a lot of the dependencies like glew depend on opengl/system still instead of the libglvnd Conan package. To figure out what's really going would require a bigger deep dive into running with some different environment variables to debug the libglvnd loading and the Mesa drivers. I don't think that's really necessary at this point though.

It's also worth noting that an outdated recipe revision of libglvnd could also be causing the problem.

@jwillikers
Copy link
Contributor

jwillikers commented May 22, 2024

I've opened #24067 to fix this. Also, @xyz1001 Do you know if you are using the latest revision of the libglvnd recipe? Just want to take sure you're not running into this due to an old recipe revision.

@xyz1001
Copy link
Contributor Author

xyz1001 commented May 23, 2024

@jwillikers Yes, I am using the latest revision of the libglvnd recipe. The conanfile.py in my local conan cache is same as that in conan-center-index repo.

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.

4 participants