On 11/21/2017 03:29 AM, Eric Engestrom wrote: > On Monday, 2017-11-20 17:24:09 -0800, Ian Romanick wrote: >> diff --git a/src/compiler/Makefile.spirv.am b/src/compiler/Makefile.spirv.am >> index 9841004..4bc684a 100644 >> --- a/src/compiler/Makefile.spirv.am >> +++ b/src/compiler/Makefile.spirv.am >> @@ -20,6 +20,14 @@ >> # FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER >> DEALINGS >> # IN THE SOFTWARE. >> >> +spirv/spirv_capabilities.cpp: spirv/spirv_capabilities_h.py >> spirv/spirv.core.grammar.json >> + $(MKDIR_GEN) >> + $(PYTHON_GEN) $(srcdir)/spirv/spirv_capabilities_h.py >> $(srcdir)/spirv/spirv.core.grammar.json $@ || ($(RM) $@; false) >> + >> +spirv/spirv_capabilities.h: spirv/spirv_capabilities_h.py >> spirv/spirv.core.grammar.json >> + $(MKDIR_GEN) >> + $(PYTHON_GEN) $(srcdir)/spirv/spirv_capabilities_h.py >> $(srcdir)/spirv/spirv.core.grammar.json $@ || ($(RM) $@; false) > > Missing `--gen-cpp`/`--gen-h`
That's weird. I made that change after it failed to build on our CI system. This also prompted me to poke at Dylan about when we can remove the autotools build system so that I only have to update 1 build instead of 2. :) I will look into this... >> + >> spirv/spirv_info.c: spirv/spirv_info_c.py spirv/spirv.core.grammar.json >> $(MKDIR_GEN) >> $(PYTHON_GEN) $(srcdir)/spirv/spirv_info_c.py >> $(srcdir)/spirv/spirv.core.grammar.json $@ || ($(RM) $@; false) _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
