Fragment shader compilation fails on init

OS platform / Browser

Fedora 31 Linux (kernel: 5.7.7) / Mozilla Firefox 78.0.1 (64 bit)

melonJS version

7.1.1 (latest) Tried with 6.4.0 -> no error

Bug description

The entire framework doesn’t start. An error occurs while creating the WebGL renderer. Apparently the compilation step of the fragment shader of the QuadGLShader fails (see error log)

Steps to reproduce the bug

Execute part 2 of the platformer tutorial (only happens on my machine, checked with some colleagues, same configuration, different gfx card)

Console log Extract

Error creating WebGL renderer :0:2(12): warning: extension `GL_ARB_gpu_shader5' unsupported in fragment shader
0:21(29): error: sampler arrays indexed with non-constant expressions are forbidden in GLSL 1.30 and later
melonjs.js:19930:20

Uncaught TypeError: _context is null
    getContext2d http://localhost:8000/lib/melon/melonjs.js:20562
    init http://localhost:8000/lib/melon/melonjs.js:21378
    Class http://localhost:8000/lib/melon/melonjs.js:193
    autoDetectRenderer http://localhost:8000/lib/melon/melonjs.js:19934
    init http://localhost:8000/lib/melon/melonjs.js:20128
    onload http://localhost:8000/js/game.js:15
    onReady http://localhost:8000/:46
    _domReady http://localhost:8000/lib/melon/melonjs.js:454
melonjs.js:20562:14

Uncaught TypeError: me.video.renderer is undefined
    init http://localhost:8000/lib/melon/plugins/debug/debugPanel.js:232
    Class http://localhost:8000/lib/melon/melonjs.js:193
    init http://localhost:8000/lib/melon/plugins/debug/debugPanel.js:52
    Class http://localhost:8000/lib/melon/melonjs.js:193
    register http://localhost:8000/lib/melon/melonjs.js:31264
debugPanel.js:232:56

[obiot]

sure yes, if it runs, and performances are as expected on your side, it’s acceptable 😃

end of next week is the maximum “deadline” for the 8.0 release anyway, so not long to wait !

[mbolis]

Thanks @obiot for the clear answer.

I ended up overwriting me.QuadGLShader and passing on an unrolled shader. Being a bit in a hurry, I believe it’s an acceptable solution.

Do you believe it’s a passable solution, while 8.0 gets to release?