Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

GLM 1.0 compatibility #721

Merged
merged 1 commit into from
Jan 28, 2024
Merged

GLM 1.0 compatibility #721

merged 1 commit into from
Jan 28, 2024

Conversation

cjmayo
Copy link
Contributor

@cjmayo cjmayo commented Jan 27, 2024

Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."

Became an error in:
g-truc/glm@f86092a#diff-b25c43cdda0d7be5dc5ea91beefb168cc1bd2522d60c7aa9b6c6164c0630e173

I haven't noticed any other effects from defining GLM_ENABLE_EXPERIMENTAL. But...

Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has
become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."
Copy link

codecov bot commented Jan 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9690305) 91.67% compared to head (59f4e6c) 91.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #721   +/-   ##
=======================================
  Coverage   91.67%   91.67%           
=======================================
  Files          37       37           
  Lines        4732     4732           
=======================================
  Hits         4338     4338           
  Misses        394      394           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pca006132
Copy link
Collaborator

Thanks!

@pca006132 pca006132 merged commit 0b7ee87 into elalish:master Jan 28, 2024
18 checks passed
@cjmayo cjmayo deleted the glm-1.0 branch January 29, 2024 20:02
cartesian-theatrics pushed a commit to SovereignShop/manifold that referenced this pull request Mar 11, 2024
Using glm/gtx/compatibility.hpp without GLM_ENABLE_EXPERIMENTAL has
become an error:

error: #error "GLM: GLM_GTX_compatibility is an experimental extension
               and may change in the future. Use #define GLM_ENABLE_EXPERIMENTAL
	       before including it, if you really want to use it."
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants