FS#77608 - [powerdevil] Demote networkmanager-qt and bluez-qt to optdepends
Attached to Project:
Arch Linux
Opened by Emil (xexaxo) - Tuesday, 21 February 2023, 22:12 GMT
Last edited by Antonio Rojas (arojas) - Wednesday, 22 February 2023, 07:58 GMT
Opened by Emil (xexaxo) - Tuesday, 21 February 2023, 22:12 GMT
Last edited by Antonio Rojas (arojas) - Wednesday, 22 February 2023, 07:58 GMT
|
Details
Description:
I realise there has been many bug reports about this, although I would ask the maintainers to read until the end before closing as duplicate. On the tracker we have many similar requests, roughly grouped in two: - "remove" networkmanager/bluez Not possible since it will break people who do use the NM and friends - demote networkmanager/bluez to optional dependency Possible today, see below for more. If you look closely at the code you'll see that the nm/bluez functionality is limited to the action, which is it's own module (on-demand loaded DSO). As such neither the executables nor the main library powerdevilcore actually link to it. Yes, currently the code claims that powerdevilcore does link [1], although that is an actual bug (over-linking) and I've sent a patch upstream [2]. I believe when the original reported came out, we did not have the automatic "as-needed" in our default linker options. Thus we were all mislead that the "executables" needed NM/Bluez. Since that change, the overlinking is automatically resolved and things just work. See reproducer below for step-by-step guide Additional info: * package version(s) powerdevil 5.27.1-1 * link to upstream bug report, if any [1] https://github.com/KDE/powerdevil/commit/829ab7996660520dd8c342c71f8d5c351c52f39b [2] https://invent.kde.org/plasma/powerdevil/-/merge_requests/136 Steps to reproduce: - install plasma and friends - pacman -R networkmanager-qt --assume-installed networkmanager-qt - reboot - start plasma -> system settings -> Power management - note that the wireless power management "page" will be missing - all other Power management pages and toggles work fine |
This task depends upon
Closed by Antonio Rojas (arojas)
Wednesday, 22 February 2023, 07:58 GMT
Reason for closing: Implemented
Additional comments about closing: powerdevil 5.27.1-2
Wednesday, 22 February 2023, 07:58 GMT
Reason for closing: Implemented
Additional comments about closing: powerdevil 5.27.1-2
Comment by Toolybird (Toolybird) -
Wednesday, 22 February 2023, 02:01 GMT
Comment by Antonio Rojas (arojas) -
Wednesday, 22 February 2023, 07:57 GMT
With your patch merged, it does indeed seem feasible now.
Nice. This has actually nothing to do with --as-needed, the
functionality was moved to a plugin in upstream commit
https://invent.kde.org/plasma/powerdevil/-/commit/3aa08ef577b295ea81387d3008c7cb02d9d8ccac