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

[msys2] Recent GLM update doesnt carry its pkg-config metadatas #7874

Closed
ylmrx opened this issue Jan 27, 2024 · 2 comments · Fixed by #7885
Closed

[msys2] Recent GLM update doesnt carry its pkg-config metadatas #7874

ylmrx opened this issue Jan 27, 2024 · 2 comments · Fixed by #7885

Comments

@ylmrx
Copy link

ylmrx commented Jan 27, 2024

The install script will pick latest packages, it seems glm package doesn't embed the relevant pc file, thus breaking the further make commands.

It builds fine rolling back to glm-0.9.9.x, like so:

wget https://repo.msys2.org/mingw/mingw64/mingw-w64-x86_64-glm-0.9.9.8-2-any.pkg.tar.zst
pacman -U mingw-w64-x86_64-glm-0.9.9.8-2-any.pkg.tar.zst --noconfirm
oxillo added a commit to oxillo/openFrameworks that referenced this issue Jan 29, 2024
As of GLM 1.0, .pc file not provided anymore. However, it was referencing standard include path.
Should work also with previous version of GLM.
oxillo added a commit to oxillo/openFrameworks that referenced this issue Feb 3, 2024
As of GLM 1.0, .pc file not provided anymore. However, it was referencing standard include path.
Should work also with previous version of GLM.
@GorgeousOne
Copy link

I just spent 6 hours trying to learn what msys2 package management is and what make files do to come to this exact conclusion 😂 but good to see that there's already a fix

@danoli3
Copy link
Member

danoli3 commented Feb 4, 2024

Yeah not great

danoli3 pushed a commit that referenced this issue Feb 4, 2024
As of GLM 1.0, .pc file not provided anymore. However, it was referencing standard include path.
Should work also with previous version of GLM.
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 a pull request may close this issue.

3 participants