guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#57297] [PATCH 0/7] Update vulkan packages and add vulkan-validation


From: Kaelyn
Subject: [bug#57297] [PATCH 0/7] Update vulkan packages and add vulkan-validationlayers
Date: Sat, 15 Oct 2022 16:52:50 +0000

Hi,

Sorry for the long period of silence on this patch set! I have finally found a 
fix for the mpv build failure with the updated spirv-headers and spirv-tools 
(and likely any others that failed with similar undefined symbols as well). 
While trying to figure out why mpv wasn't building when other distros have been 
shipping the same version of mpv against updated spirv & vulkan packages, I 
started examining their package definitions to see why that may be.

The fix is to add "-DSPIRV_TOOLS_BUILD_STATIC=OFF" to the configure flags for 
spirv-tools. Apparently a change to spirv-tools in late 2020 caused issues when 
statically linking (lost the link to the ML thread), and the common packaging 
approach seems to be to disable the static libs with that flag[1][2].

Quick question on procedure: would it be better to send to this bug a revised 
version of the single patch (#2 of 7), or to rebase the set against the tip of 
the staging branch and send in a v3 of all 7 patches?

Cheers,
Kaelyn


[1] 
https://github.com/archlinux/svntogit-packages/blob/packages/spirv-tools/trunk/PKGBUILD#L27
[2] 
https://gitweb.gentoo.org/repo/gentoo.git/tree/dev-util/spirv-tools/spirv-tools-1.3.224.ebuild#n39






reply via email to

[Prev in Thread] Current Thread [Next in Thread]