Bug 241490 - x11-toolkits/qt5-gui: requires runtime dependency on mesa-dri
Summary: x11-toolkits/qt5-gui: requires runtime dependency on mesa-dri
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Adriaan de Groot
Depends on:
Blocks: 241485
  Show dependency treegraph
Reported: 2019-10-25 14:15 UTC by Guido Falsi
Modified: 2022-02-22 23:29 UTC (History)
6 users (show)

See Also:
bugzilla: maintainer-feedback? (kde)

adding mesa-dri dependency explicitly. (733 bytes, patch)
2019-10-25 14:15 UTC, Guido Falsi
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Guido Falsi freebsd_committer 2019-10-25 14:15:54 UTC
Created attachment 208597 [details]
adding mesa-dri dependency explicitly.


A user of the calibre port is launching calibre from a remote machine where he is not installing the full Xorg stack, but only calibre and it's direct dependencies.

He reported in bug #241485 that in this setup the calibre ebook-reader component fails to start properly. Installing mesa-dri fixed the issue for him, letting the calibre component find the swrast driver it was needing.

I have reproduced his findings using to VirtualBox VMs, so I guessed qt5-gui is the port whose libraries are actually making the GL calls depending on the mesa-dri drivers.

As a side note my machine has an nvidia card and in this combination things don't work anyway, I guess that's because the nvidia dirver overrides libGL and friends.

Do you think it is correct to add this run time dependency? For most users installing qt5 on the machine together with the Xserver this is a noop. It could help in other situations.

Comment 1 Guido Falsi freebsd_committer 2019-11-18 14:40:47 UTC
Thinking again about it, I suspect the dependency should be added to mesa-libs or through USES=gl.

Any thought about this?
Comment 2 commit-hook freebsd_committer 2021-02-20 18:32:27 UTC
A commit references this bug:

Author: adridg
Date: Sat Feb 20 18:31:32 UTC 2021
New revision: 566177
URL: https://svnweb.freebsd.org/changeset/ports/566177

  Add mesa-dri dependency for qt5-gui

  There are cases (headless, no xorg-server) where a qt5-gui application
  still needs to run. It can use swrast, but only if that is installed.
  Add the mesa-dri dependency to make that happen automatically.

  For "normal" Qt5 usage, that is, Qt5 on the desktop, xorg-server already
  pulls this in. The only change is for systems with no X11, no Wayland,
  but with Qt5 GUI applications.

  PR:		241490
  Submitted by:	madpilot
  Reported by:	madpilot

Comment 3 Adriaan de Groot freebsd_committer 2021-02-20 18:34:44 UTC
I've gone and landed your original suggestion. I suppose a USE_GL=sw might have done as well, although we'd need to introduce that in `gl.mk` first. This is the simpler approach.
Comment 4 Guido Falsi freebsd_committer 2021-02-20 19:05:54 UTC
(In reply to Adriaan de Groot from comment #3)

Thanks! This fixed a long standing issue about qt applications usage on remote systems.
Comment 5 VVD 2021-02-20 19:48:49 UTC
(In reply to Guido Falsi from comment #4)
But it now add unnecessary fat dependency for headless installations.
Why not optional?
Comment 6 Guido Falsi freebsd_committer 2021-02-20 20:03:35 UTC
(In reply to VVD from comment #5)

It could also be an option, but it should be enabled by default, otherwise the problem would not be fixed for people using binary packages.

Also, the "fatness" is 51.9 MiB according to pkg on my laptop (amd64). Is that really relevant for modern systems with extremely cheap disk space? (On virtual hosting too)
Comment 7 Jan Beich freebsd_committer 2021-02-20 20:38:54 UTC
(In reply to Guido Falsi from comment #6)
> Also, the "fatness" is 51.9 MiB according to pkg on my laptop (amd64).

Did you miss the elephant dependency? review D16457 is still far away.

$ pkg install -qy qt5-gui
$ pkg install -n mesa-dri
Updating FreeBSD repository catalogue...
FreeBSD repository is up to date.
All repositories are up to date.
The following 7 package(s) will be affected (of 0 checked):

New packages to be INSTALLED:
        libXv: 1.0.11_2,1
        libXvMC: 1.0.12
        libedit: 3.1.20191231,1
        llvm10: 10.0.1_5   <------------ THIS
        lua52: 5.2.4
        mesa-dri: 20.2.3_1
        perl5: 5.32.1_1

Number of packages to be installed: 7

The process will require 759 MiB more space.
124 MiB to be downloaded.
Comment 8 Guido Falsi freebsd_committer 2021-02-20 20:55:52 UTC
(In reply to Jan Beich from comment #7)

I may have not made my question clear.

The only relevant question was about the option being disabled by default, as in would you be satisfied by mesa-dri being optional but enabled by default?

I can create and propose a patch to add an option, but only if it is acceptable to have it enabled by default. Otherwise (options disabled by default) it would be as useful as not having the dependency at all for 99% of the situations I created this patch to fix.

The second part about disk sizes was irrelevant to any decision, but you replied to that one only. Your objection would not persuade me anyway, but I'm not going to fight on this. As I said above I'm willing to create a patch to make this an option, with conditions described above.
Comment 9 Adriaan de Groot freebsd_committer 2021-02-20 22:29:36 UTC
There's another way about this, and that is pkg-message (which would say "if you intend to use qt5-gui headless, install mesa-dri"). I'd forgotten about monsters like llvm.

However, the point is this:
- without mesa-dri, a Qt5-gui-using application can't run the GUI (depending a little on which code paths it takes, I'm sure)
- on the desktop, other things will pull in mesa-dri anyway

So it comes down to: are uses of qt5-gui where an application (that uses qt5-gui) does *not* need mesa-dri? The original bug report has an n=1 incidence of "no". Perhaps futzing with the QPA will do it (e.g. set the QPA to use offscreen output; in which case I'd return to the pkg-message approach: at some point you have to trust the user to put the parts together that cannot be guessed)
Comment 10 Guido Falsi freebsd_committer 2021-02-20 22:33:54 UTC
(In reply to Adriaan de Groot from comment #9)

I would have no objection to the package message approach.

My original submission was a suggestion of a possible solution, which looked reasonable at the time, and also to spur discussion. I'm not "married" to the unconditional dependency approach.
Comment 11 Adriaan de Groot freebsd_committer 2021-02-22 15:21:25 UTC
The linked bug is shiboken falling over when there's a newer llvm installed; that is exposed by this change, but should be fixed independently of what this change does.

@VVD: while this is a "fat" dependency, the question is: is there **any** way of using an application, which uses qt5-gui, on a headless system, without that dependency? If there is, I'll roll back and shift blame to the user with a pkg-message. If there isn't, then stripping the fat just means "this won't work". So I need a specific example.
Comment 12 VVD 2021-02-22 19:02:22 UTC
(In reply to Adriaan de Groot from comment #11)
emulators/virtualbox-ose work without mesa-dri installed via ssh -Y.
Comment 13 Dries Michiels freebsd_committer 2021-03-02 13:09:08 UTC
I think qbittorrent@nox also uses qt5-gui, but is actually perfectly accessible through webui.
Comment 14 Kubilay Kocak freebsd_committer freebsd_triage 2022-02-22 23:29:36 UTC
^Triage: ports r566177 resolved the issue 'as reported', last comment is over a year old, assign to committer that resolved, and close.

Please create a new issue (referencing this one in See Also), for any remaining issues, improvements or proposed changes