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

solve issue #1780 (built-in matdefs/shaders lack core-profile support) #1781

Merged
merged 6 commits into from
Mar 11, 2022

Conversation

stephengold
Copy link
Member

Added "GLSL150" to matdefs and #import "Common/ShaderLib/GLSLCompat.glsllib" to shaders. Not certain I got them all.

@stephengold stephengold added this to the Future Release milestone Mar 9, 2022
@stephengold stephengold linked an issue Mar 9, 2022 that may be closed by this pull request
@stephengold
Copy link
Member Author

I verified that this PR solves issue 1780 on macOS Monterey.
In my opinion this would be worth including in JME 3.5.1 if we decide to go that route.

@stephengold
Copy link
Member Author

Optimistically interpreting the thumbs-up emojis as positive code reviews, I plan to self-integrate this PR in about 24 hours ... unless there's something to discuss.

@stephengold stephengold merged commit c648aad into master Mar 11, 2022
@stephengold stephengold deleted the sgold/issue-1780 branch March 11, 2022 20:41
stephengold added a commit that referenced this pull request Mar 18, 2022
#1781)

* jme3-terrain:  add GLSLCompat.glsllib to 4 jme3-terrain frag shaders

* jme3-terrain:  add GLSLCompat.glsllib to 4 jme3-terrain vertex shaders

* jme3-terrain:  add GLSL150 to 3 jme3-terrain material definitions

* jme3-vr:  add GLSLCompat.glsllib to 5 shaders

* jme3-niftygui:  add GLSLCompat.glsllib to 6 shaders

* jme3-niftygui:  add GLSL150 to 3 material definitions
@stephengold stephengold modified the milestones: Future Release, v3.5.1 Mar 18, 2022
@stephengold stephengold added the bug Something that is supposed to work, but doesn't. More severe than a "defect". label Mar 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that is supposed to work, but doesn't. More severe than a "defect".
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RendererException in TerrainLighting.frag
1 participant