search

conda-forge / pyopengl-feedstock

A conda-smithy repository for pyopengl.
BSD 3-Clause "New" or "Revised" License
2 stars 11 forks source link

error on env activation #14

Closed beckermr closed 2 years ago

beckermr commented 5 years ago

Hi @conda-forge/pyopengl!

In a recipe rebuild here, https://github.com/conda-forge/xpdsim-feedstock/pull/6, there is an error on environment activation from this package. Can one you all investigate?

almarklein commented 5 years ago

Just made a PR to push a new version and rerender. Maybe that will help. Do you happpen to know why CI does not kick in for that PR (I have not been very active with conda-forge lately).

almarklein commented 5 years ago

Do you happpen to know why CI does not kick in for that PR [...]?

Nevermind

almarklein commented 5 years ago

PR merged, the new build is available on anaconda.org

CJ-Wright commented 5 years ago

It seems that the new version has not fixed the issue conda.CondaMultiError: post-link script failed for package conda-forge::pyopengl-3.1.3b2-py_0

almarklein commented 5 years ago

It looks like it's supposed to fail on some systems: https://github.com/conda-forge/pyopengl-feedstock/blob/master/recipe/post-link.sh

beckermr commented 5 years ago

So this is a change in the CI environment? Conda depending on stuff out of itself is a necessary evil...

CJ-Wright commented 5 years ago

Yea something tells me we can ship our own drivers.

almarklein commented 5 years ago

Would it help if we print the warning and then exit 0 instead?

beckermr commented 5 years ago

Not if it leaves the env broken. We probably need to package the driver if possible.

CJ-Wright commented 5 years ago

Maybe have a CDT? Currently I'm trying to mitigate this by removing this package where it is an optional dependency in my tree.

beckermr commented 5 years ago

Hrrmmmmm. Anaconda docs indicate that the CDT for OpenGL is under discussion. I think we have hit a thing that is already a thing.

https://conda.io/docs/user-guide/tasks/build-packages/compiler-tools.html#cdt-packages

jaimergp commented 2 years ago

Closed by #21