Not providing a path allows the ICD to work on multi-arch systems but breaks it if you install anywhere other than /usr/lib. Given that users may be installing locally in .local or similar, we probably do want to provide a filename. Distros can carry a revert of this commit if they want an intel_icd.json file without the path.
Signed-off-by: Jason Ekstrand <ja...@jlekstrand.net> Cc: Mark Janes <mark.a.ja...@intel.com> --- src/intel/vulkan/.gitignore | 1 + src/intel/vulkan/Makefile.am | 7 ++++++- src/intel/vulkan/{intel_icd.json => intel_icd.json.in} | 2 +- 3 files changed, 8 insertions(+), 2 deletions(-) rename src/intel/vulkan/{intel_icd.json => intel_icd.json.in} (59%) diff --git a/src/intel/vulkan/.gitignore b/src/intel/vulkan/.gitignore index bde5cd8..a099ff6 100644 --- a/src/intel/vulkan/.gitignore +++ b/src/intel/vulkan/.gitignore @@ -3,3 +3,4 @@ /anv_entrypoints.h /anv_timestamp.h /dev_icd.json +/intel_icd.json diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am index ad0148d..9fef960 100644 --- a/src/intel/vulkan/Makefile.am +++ b/src/intel/vulkan/Makefile.am @@ -141,7 +141,7 @@ anv_timestamp.h: $(AM_V_GEN) echo "#define ANV_TIMESTAMP \"$(TIMESTAMP_CMD)\"" > $@ BUILT_SOURCES = $(VULKAN_GENERATED_FILES) -CLEANFILES = $(BUILT_SOURCES) dev_icd.json +CLEANFILES = $(BUILT_SOURCES) dev_icd.json intel_icd.json EXTRA_DIST = \ $(top_srcdir)/include/vulkan/vk_icd.h \ anv_entrypoints_gen.py \ @@ -170,6 +170,11 @@ dev_icd.json : dev_icd.json.in -e "s#@build_libdir@#${abs_top_builddir}/${LIB_DIR}#" \ < $(srcdir)/dev_icd.json.in > $@ +intel_icd.json : intel_icd.json.in + $(AM_V_GEN) $(SED) \ + -e "s#@install_libdir@#${libdir}#" \ + < $(srcdir)/intel_icd.json.in > $@ + # Libvulkan with dummy gem. Used for unit tests. libvulkan_test_la_SOURCES = $(VULKAN_GEM_STUB_FILES) libvulkan_test_la_LIBADD = $(VULKAN_LIB_DEPS) -lX11-xcb diff --git a/src/intel/vulkan/intel_icd.json b/src/intel/vulkan/intel_icd.json.in similarity index 59% rename from src/intel/vulkan/intel_icd.json rename to src/intel/vulkan/intel_icd.json.in index 277c14e..d9b363a 100644 --- a/src/intel/vulkan/intel_icd.json +++ b/src/intel/vulkan/intel_icd.json.in @@ -1,7 +1,7 @@ { "file_format_version": "1.0.0", "ICD": { - "library_path": "libvulkan_intel.so", + "library_path": "@install_libdir@/libvulkan_intel.so", "abi_versions": "1.0.3" } } -- 2.5.0.400.gff86faf _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev