Glsl150ShaderGenerator should include GLSLCompat for GLES3

[riccardobl]

Glsl150ShaderGenerator should include GLSLCompat in the shaders, or otherwise manually set the default precisions otherwise shadernodes for GLES3 will fail compilation with

com.jme3.renderer.RendererException: compile error in: ShaderSource[name=Default.frag, defines, type=Fragment, language=GLSL300]
ERROR: 0:11: '' : No precision specified for (float)
ERROR: 0:13: '' : No precision specified for (float)
ERROR: 0:14: '' : No precision specified for (float)
ERROR: 0:15: '' : No precision specified for (float)
 ...

[riccardobl]

Workaround

1. add this patched generator to the app

package com.animotiv.designertest.system;

import com.jme3.asset.AssetManager;
import com.jme3.material.ShaderGenerationInfo;
import com.jme3.shader.Glsl300ShaderGenerator;
import com.jme3.shader.Shader;


public class GLES300ShaderGenerator extends Glsl300ShaderGenerator {

    public GLES300ShaderGenerator(AssetManager assetManager) {
        super(assetManager);
    }

    protected void generateUniforms(StringBuilder source, ShaderGenerationInfo info, Shader.ShaderType type) {
        source.append( "#ifdef GL_ES\n" +
                "  #ifdef FRAGMENT_SHADER\n" +
                "    precision highp float;\n" +
                "    precision highp int;\n" +
                "    #if __VERSION__ >= 130\n" +
                "      precision highp sampler2DArray;        \n" +
                "    #endif\n" +
                "    precision highp sampler2DArray;\n" +
                "    precision highp sampler2DShadow;\n" +
                "    precision highp samplerCube;\n" +
                "    precision highp sampler3D;\n" +
                "    precision highp sampler2D;\n" +
                "    #if __VERSION__ >= 310\n" +
                "      precision highp sampler2DMS;\n" +
                "    #endif\n" +
                "  #endif\n" +
                "#endif\n");
        super.generateUniforms(source, info, type);
    }
}

2. pass it in the assetManager at the beginning of simpleInitApp

        assetManager.setShaderGenerator(new GLES300ShaderGenerator(assetManager));

note: this is not a very pretty solution, if possible it should be fixed properly in the engine

[stephengold]

@riccardobl Thank you for documenting the issue and the workaround.

[github-actions]

💰 This issue has a bounty

Resolve it to receive a reward.
For details (amount, rules, eligibility), see: https://hub.jmonkeyengine.org/t/bounty-program-trial-starts-today/49394/

If you want to start working on this, comment on this issue with your intent.
If accepted by a maintainer, the issue will be assigned to you.