Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not install vulkan headers, but add imx_icd so it loads libvulkan_VSI so it doesn't conflict with libvulkan-dev #5

Merged

Conversation

charles2910
Copy link
Contributor

@charles2910 charles2910 commented Nov 7, 2024

@charles2910 charles2910 self-assigned this Nov 7, 2024
@leonheldattoradex
Copy link
Contributor

Might be good to add the meta-freescale reference somewhere in this MR/commit history.

debian/rules Show resolved Hide resolved
- d/extra_files/imx_icd.json: add file from meta-freescale to tell
  vulkan-loader to use libvulkan_VSI provided by imx-gpu-viv-wayland;
- d/imx-gpu-viv-wayland.install: install imx_icd.json;
- d/rules: sed imx_icd.json with correct values and drop vulkan headers
  as done in meta-freescale [1].

[1] Freescale/meta-freescale@6b4da0b

Related-to: TCCP-851
Signed-off-by: Carlos Henrique Lima Melara <carlos.melara@toradex.com>
@charles2910 charles2910 force-pushed the charles/update-vulkan-install branch from 1b31a86 to 0dc8e42 Compare November 8, 2024 13:35
@charles2910
Copy link
Contributor Author

Might be good to add the meta-freescale reference somewhere in this MR/commit history.

Added to MR and commit history just to be safe.

@charles2910 charles2910 force-pushed the charles/update-vulkan-install branch from 0dc8e42 to 76a85dd Compare November 8, 2024 13:37
@leonheldattoradex leonheldattoradex merged commit e7e0677 into toradex/scarthgap/bookworm Nov 10, 2024
1 check passed
@leonheldattoradex leonheldattoradex deleted the charles/update-vulkan-install branch November 10, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants