vispy / vispy / 18319040557

75%

push

github

Revert pyopengl pin with `LD_PRELOAD` workaround (#2702)

This reverts the pyopengl pin as in #2673 but adds a workaround that let
CI pass in my fork.

As best I can understand the issue, pyopengl changed slightly the way it
resolves shared libraries in
https://github.com/mcfletch/pyopengl/pull/93 (v3.1.7+), which conflicts
with how `sdl2` loads the same libraries or expects to find symbols in
them...somehow. I can look into an upstream PR to pyopengl to resolve
this, but that will take some time and I'm not super confident about
making changes there (this is pretty convoluted for me to test).

This PR adds `LD_PRELOAD` (which I admit to not fully understanding) to
change which symbols are prioritized when pyopengl loads openGL
libraries.

I don't think anyone has reported these issues outside of CI, so I think
this workaround is reasonable. If anyone reports such a bug with local
usage, hopefully this is a practical workaround for them as well. I'm
not sure how often people are using the SDL2 backend anyway, but this
feels better than the pyopengl pins/exlusions to me.

5873 of 10620 branches covered (55.3%)

24195 of 32302 relevant lines covered (74.9%)

2.46 hits per line