Missing libxnvctrl package causes crash

Description:

MangoHUD crashes when the libxnvctrl package is not installed, as soon as the package is installed MangoHUD starts normally.

Probably caused by the lack of the -Dwith_xnvctrl=disabled argument (README MangoHUD).

I believe that without this argument you would have to make the libxnvctrl package a dependency.

Btw, a user gave me feedback that version 0.8.0 is working correctly on AMD cards, it seems to me that the crash is only caused in the scenario of the presence of an NVIDIA GPU and the lack of the libxnvctrl package.

Additional info:

• package version(s): 0.8.0-1
• Kernel Version: 6.13.2-zen1-1-zen (64-bit)
• Graphics Platform: Wayland
• Graphics Processor: NVIDIA GeForce GTX 1050 Ti

Steps to reproduce:

1. Have an NVIDIA GPU
2. Do not have the libxnvctrl package installed.
3. Use MangoHUD with some application ($ mangohud vkcube)
4. Get a crash

added priority3-normal severity4-low statusunconfirmed labels

assigned to @freswa, @toolybird, and @gromit

changed title from Absence of libxnvctrl package causes crash to Missing libxnvctrl package causes crash

So you're saying the optional dep description needs to change? I guess it could be more explicit, but the dep is there.

Not necessarily, I compiled the package with the -Dwith_xnvctrl=disabled argument and was able to use it without needing to have libxnvctrl installed. It's more of a report than a suggestion of what should be done, I don't know what the best approach would be.

Thanks for the report. Note: while it does now run after installing libxnvctrl, it still segfaults for me upon exit.

added scopeenhancement statusconfirmed labels and removed statusunconfirmed label

assigned to @grawlinson and @serebit and unassigned @freswa, @toolybird, and @gromit

Ah, yep, I tested the release on AMD which is why I didn't see this. Fixing now.

closed with commit bfa2f05b

Actually the issue is MangoHUD not handling the libxnvctrl load failure properly: https://github.com/flightlessmango/MangoHud/commit/fde62484e0e1724569ecc86df8e1ffbe2bc3510a (so this commit should be backported instead of requiring a strange X11-dependent library)

Also the 32-bit build of MangoHUD didn't receive the fix at all (so I'll make another issue there)

added resolutioncompleted label

mentioned in issue lib32-mangohud#1 (closed)

mentioned in issue #2 (closed)