search

AtomLinter / linter-glsl

Atom package that lints GLSL shaders on the fly.
https://atom.io/packages/linter-glsl
MIT License
15 stars 6 forks source link

Stop using glslangValidator? #71

Closed fand closed 6 years ago

fand commented 6 years ago

glslangValidator makes it difficult to install linter-glsl. It would be great if we can install linter-glsl without any prerequisites.

As Atom is built on Electron, we can validate GLSL shaders by compiling them background. Many online GLSL editors have their own validators implemented in this way, such as GLSL Sandbox, Shadertoy, etc.

Arcanemagus commented 6 years ago

GLSL Sandbox appears to have a very basic parser (here). Shadertoy doesn't seem to publish their source.

So it sounds like you are asking for a completely new linter to be written? This is just a wrapper around glslangValidator... the reference validator from the spec itself.

If you find another linter for GLSL code feel free to write a wrapper to bring it into the Atom linter ecosystem, I'd be happy to have it in the organization if you want too 😉.