Community Packages

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#73506 - [wlroots] WLR_RENDERER=vulkan requires vulkan-validation-layers

Attached to Project: Community Packages
Opened by m1s4k1 (m1s4k1) - Wednesday, 26 January 2022, 06:22 GMT
Last edited by Toolybird (Toolybird) - Sunday, 13 November 2022, 05:31 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Brett Cornwall (ainola)
Maxim Baz (maximbaz)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
Currently vulkan-validation-layers is required to make WLR_RENDERER=vulkan work. Should it be added as optional dependency?

upstream issue: https://gitlab.freedesktop.org/wlroots/wlroots/-/issues/3283

upstream MR: https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/3318

Additional info:
* package version(s): wlroots 0.15.0-4; sway 1:1.7-1
This task depends upon

Closed by  Toolybird (Toolybird)
Sunday, 13 November 2022, 05:31 GMT
Reason for closing:  Fixed
Additional comments about closing:  Reporter says "Fixed in upstream. https://gitlab.freedesktop.org/wlroots/w lroots/-/merge_requests/3850"
Comment by Maxim Baz (maximbaz) - Wednesday, 16 February 2022, 18:51 GMT
Thanks for letting us know! I'm not entirely sure it's a good idea to add optdepends, validation layers is a huge and heavy dependency meant only for debugging purposes, it is not correct for wlroots to require it... I get it that it's a temporary dependency while the feature is very unstable, and indeed we enabled the feature on purpose, to make it easier for users to tinker around if they want and know what they are doing, but at the same time adding debugging dependencies to the package seems a bit wrong, if you see what I mean... Brett, any opinions? Feel free to add if you think it should be done, in that case lets just remember to remove it.
Comment by Brett Cornwall (ainola) - Thursday, 17 February 2022, 01:13 GMT
I definitely don't think we should add a full dependency, especially since it won't be a dep in the long term. Considering the unstable nature of the Vulkan renderer I think it would be best for us to not parade the renderer as an optdep like it's a beta feature. That it's for debugging purposes also disqualifies it as a candidate IMO (by that logic we'd optdepends e.g. gdb everywhere). But then again, we typically don't ship software that requires gdb to run. :D

As a middleground, we could update the sway wiki page, but I'm not so sure it would belong there: It feels like it should be mentioned upstream in their Vulkan documentation for contributing (à la "If you're on Ubuntu, install `foo`. If on Arch Linux, install vulkan-validation-layers).

Loading...