Revert "glsl: Refuse to link GLSL 1.40+ shaders that would use fixed function."
authorEric Anholt <eric@anholt.net>
Mon, 16 Apr 2012 20:37:04 +0000 (13:37 -0700)
committerEric Anholt <eric@anholt.net>
Mon, 16 Apr 2012 20:50:05 +0000 (13:50 -0700)
commit35f2fb70d3826df70fa2f53af05339137cddefc8
tree01589fe2053c8c6e4175e46aee2ab56675f3820c
parentf084b5ad323f021e9a0f667b977942fb25f69c43
Revert "glsl: Refuse to link GLSL 1.40+ shaders that would use fixed function."

This reverts commit 4ec449a6ed1d2cea3bf83d6518b3b352ce5daceb.

I meant to not push this one.  Review found that a link error is not
mandated: it should link, but you get undefined rendering if you rely
on a missing stage.

     page 42/55 section 2.11 "Vertex Shaders":

     "If the program object has no vertex shader, or no program object
      is currently in use, the results of vertex shader execution are
      undefined."

(and similar for page 160/173 section 3.9 "Fragment Shaders" for FS,
and page 45/58 section 2.11.2 "Program Objects" for program being 0)

It turns out the commit was broken anyway, because it was missing a
"goto done", so linkstatus got smashed back to true later and the
error just showed up as a warning in the infolog.
src/glsl/linker.cpp