Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 715528 - dev-qt/qtgui-5.14.1 ignores accessibility use flag (PATCH)
Summary: dev-qt/qtgui-5.14.1 ignores accessibility use flag (PATCH)
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords: PATCH
Depends on: 715774
Blocks:
  Show dependency tree
 
Reported: 2020-03-30 21:42 UTC by Dennis Nezic
Modified: 2020-04-01 19:12 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dennis Nezic 2020-03-30 21:42:50 UTC
Tiny little fix. The current ebuild (up to 5.14.1) doesn't respect the "accessibility" USE flag ... ie. it gets compiled regardless, even if I disable the USE flag.

--- /dev/shm/share/fuck 2020-03-30 17:29:32.372534826 -0400
+++ ./qtgui-5.14.1-r3.ebuild    2020-03-30 17:29:16.000000000 -0400
@@ -165,6 +165,7 @@
 src_configure() {
        local myconf=(
                $(usex dbus -dbus-linked '')
+               $(qt_use accessibility)
                $(qt_use egl)
                $(qt_use eglfs)
                $(usex eglfs '-gbm -kms' '')

Reproducible: Always
Comment 1 Davide Pesavento gentoo-dev 2020-04-01 16:37:24 UTC
NACK. Upstream strongly recommends keeping accessibility enabled. We used to allow disabling it in Qt4 and it caused countless problems. I don't want to support this configuration in gentoo. See also https://github.com/gentoo/gentoo/blob/master/eclass/qt5-build.eclass#L541-L543
Comment 2 Andreas Sturmlechner gentoo-dev 2020-04-01 16:41:12 UTC
Should we drop the flag then completely?
Comment 3 Andreas Sturmlechner gentoo-dev 2020-04-01 16:45:12 UTC
Current use-revdeps are media-sound/picard, media-sound/teamspeak-client.
Comment 4 Davide Pesavento gentoo-dev 2020-04-01 16:48:18 UTC
The flag is there to make sure that if you want a functional accessibility support, the right dependencies are pulled in for the atspi bridge, which requires dbus. TBH the flag is somewhat historical, because in the early Qt5 days atspi-bridge required additional external deps which I didn't want to force on everyone. Those additional deps have since been dropped by upstream, but you still need to build Qt with dbus support (and the xcb platform backend).

Do you have any better ideas on how to handle this?
Comment 5 Andreas Sturmlechner gentoo-dev 2020-04-01 17:18:06 UTC
(In reply to Davide Pesavento from comment #4)
> Do you have any better ideas on how to handle this?
Probably not, after all it is a global use flag so having it has some merit. I notice it still ticks

> QT5_GENTOO_CONFIG=(
> 	accessibility:accessibility-atspi-bridge
> )

which does not appear to do any harm for a long time and even makes this build time effective.
Comment 6 Andreas Sturmlechner gentoo-dev 2020-04-01 17:23:08 UTC
Disabling it via qmake build flag definitely leads to numerous issues down the road in revdeps such as qtdeclarative, kdeclarative, PyQt5 and who knows what else, so it would require major testing work, that is just not worth the effort.
Comment 7 Davide Pesavento gentoo-dev 2020-04-01 19:11:02 UTC
(In reply to Andreas Sturmlechner from comment #6)
> Disabling it via qmake build flag definitely leads to numerous issues down
> the road in revdeps such as qtdeclarative, kdeclarative, PyQt5 and who knows
> what else, so it would require major testing work, that is just not worth
> the effort.
+1