Skip to content

Fix issue with PyOpenGL-accelerate on MacOS x86_64 #777

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

Conversation

cedrik-fuoco-adsk
Copy link
Contributor

@cedrik-fuoco-adsk cedrik-fuoco-adsk commented May 15, 2025

Fix issue with PyOpenGL-accelerate on MacOS x86_64

Linked issues

n/a

Summarize your change.

This PR adds a temporary changes to build PyOpenGL-accelerate from source on MacOS x86_64. The source is patched with the changes from a pending PR in the PyOpenGL repository.

Describe the reason for the change.

Cython was updated to version 3.1.0+ on May 8, causing compatibility issues with PyOpenGL-accelerate on macOS x86_64. Until a pending PR is merged in the official PyOpenGL repository, PyOpenGL-accelerate will be built from source on macOS x86_64.

Describe what you have tested and on which operating system.

Add a list of changes, and note any that might need special attention during the review.

If possible, provide screenshots.

@cedrik-fuoco-adsk cedrik-fuoco-adsk force-pushed the fix-ci-macos branch 14 times, most recently from 6ee18d0 to 1dd1751 Compare May 22, 2025 17:59
@cedrik-fuoco-adsk cedrik-fuoco-adsk force-pushed the fix-ci-macos branch 6 times, most recently from f1050ef to 5b38de2 Compare May 23, 2025 12:28
@cedrik-fuoco-adsk cedrik-fuoco-adsk force-pushed the fix-ci-macos branch 4 times, most recently from 638718c to 12b8653 Compare May 23, 2025 13:26
@cedrik-fuoco-adsk cedrik-fuoco-adsk changed the title CI - Fix for latest issue with MacOS (tentative for now, draft PR) Fix issue with PyOpenGL-accelerate on MacOS x86_64 May 23, 2025
@cedrik-fuoco-adsk cedrik-fuoco-adsk marked this pull request as ready for review May 23, 2025 14:44
Copy link
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

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

LGTM

@cedrik-fuoco-adsk cedrik-fuoco-adsk merged commit 36a92db into AcademySoftwareFoundation:main May 23, 2025
20 checks passed
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