On 22 February 2017 at 23:35, Dylan Baker <dy...@pnwbakers.com> wrote: > This changes the python generator to write the files itself, rather than > piping them out. This has a couple of advantages: first, it encapsulates > the encoding. Second, it ensures that the header file and code file are > generated at the same time with the same data. > Last time there were some serious objections to doing this. Glad that you got those covered ;-)
> Signed-off-by: Dylan Baker <dylanx.c.ba...@intel.com> > --- > src/intel/vulkan/Makefile.am | 7 ++---- > src/intel/vulkan/anv_entrypoints_gen.py | 38 > ++++++++++++--------------------- > 2 files changed, 16 insertions(+), 29 deletions(-) > > diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am > index 5a0e4ef4ff..68bc5ccf86 100644 > --- a/src/intel/vulkan/Makefile.am > +++ b/src/intel/vulkan/Makefile.am > @@ -145,11 +145,8 @@ libvulkan_intel_la_SOURCES = $(VULKAN_GEM_FILES) > > vulkan_api_xml = $(top_srcdir)/src/vulkan/registry/vk.xml > > -anv_entrypoints.h : anv_entrypoints_gen.py $(vulkan_api_xml) > - $(AM_V_GEN)$(PYTHON2) $(srcdir)/anv_entrypoints_gen.py header > $@ > - > -anv_entrypoints.c : anv_entrypoints_gen.py $(vulkan_api_xml) > - $(AM_V_GEN)$(PYTHON2) $(srcdir)/anv_entrypoints_gen.py code > $@ > +anv_entrypoints.h anv_entrypoints.c: anv_entrypoints_gen.py $(vulkan_api_xml) > + $(AM_V_GEN)$(PYTHON2) $(srcdir)/anv_entrypoints_gen.py > Please pass a "-o $dst_dir" type of argument. It will be/is broken otherwise. Small suggestion to update the Android.mk file. Feel free to Cc Tapani/Mauro so that they can test/address any concerns. Mauro, Tapani, Afaict earlier series [1] have broken things on your end, but do suggest some fixes that one can squash with this batch. Thanks Emil [1] e9dcb17962f7e58a81c93bae7bd33885675b1043 8e03250fcf4fc5de31e92ca4919959d932888a69 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
